[PD-dev] [ pure-data-Bugs-3518344 ] iemgui's Tcl code does not handle font names with spaces

[SourceForge.net]

Bugs item #3518344, was opened at 2012-04-15 20:23
Message generated for change (Tracker Item Submitted) made by eighthave
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=3518344&group_id=55736

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: externals
Group: v0.43
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: IOhannes m zmölnig (zmoelnig)
Summary: iemgui's Tcl code does not handle font names with spaces

Initial Comment:
Many font names have spaces, like "Andale Mono" or "DejaVu Mono".  The Tk font triplet expects three elements, so the font name needs to be quoted to handle font names with spaces.  So code like this:

  -font {%s %d bold}

should become:

 - font {{%s} %d bold}

Here are all the examples of this problem I found:
./iem/iemgui/src/iem_vu.c:    -font {%s %d bold} -fill #%6.6x -tags %lxLABEL\n",
./iem/iemgui/src/iem_vu.c:    sys_vgui(".x%lx.c itemconfigure %lxLABEL -font {%s %d bold} -fill #%6.6x -text {%s} \n",
Binary file ./iem/iemgui/src/iem_vu.pd_darwin matches
./iem/iemgui/src/numberbox_matrix.c:        -font {%s %d bold} -fill %s -tags %lxNUMBER_%d_%d\n", canvas, xx, yy,
Binary file ./iem/iemgui/src/numberbox_matrix.pd_darwin matches
./iem/iemgui/src/room_sim_2d.c:    -font {times %d bold} -fill #%6.6x -tags %lxSRC%d\n",
./iem/iemgui/src/room_sim_2d.c:    sys_vgui(".x%lx.c itemconfigure %lxSRC%d -font {times %d bold}\n", canvas, x, i, fs);
Binary file ./iem/iemgui/src/room_sim_2d.pd_darwin matches
./iem/iemgui/src/room_sim_3d.c:      -font {times %d bold} -fill #%6.6x -tags %lxSRC%d\n",
./iem/iemgui/src/room_sim_3d.c:    sys_vgui(".x%lx.c itemconfigure %lxSRC%d -font {times %d bold}\n", canvas, x, i, fsi);
./iem/iemgui/src/room_sim_3d.c:      sys_vgui(".x%lx.c itemconfigure %lxSRC%d -font {times %d bold}\n", canvas, x, sel, fsi);
./iem/iemgui/src/room_sim_3d.c:      sys_vgui(".x%lx.c itemconfigure %lxSRC%d -font {times %d bold}\n", canvas, x, sel, fsi);
./iem/iemgui/src/room_sim_3d.c:    sys_vgui(".x%lx.c itemconfigure %lxSRC%d -font {times %d bold}\n", canvas, x, i, fsi);
./iem/iemgui/src/room_sim_3d.c:    sys_vgui(".x%lx.c itemconfigure %lxSRC%d -font {times %d bold}\n", canvas, x, i, fsi);
Binary file ./iem/iemgui/src/room_sim_3d.pd_darwin matches
./iem/iemgui/src/sym_dial.c:      -font {%s %d bold} -fill #%6.6x -tags %lxLABEL\n",
./iem/iemgui/src/sym_dial.c:        -font {%s %d bold} -fill #%6.6x -tags %lxSYMBOL\n",
./iem/iemgui/src/sym_dial.c:  sys_vgui(".x%lx.c itemconfigure %lxLABEL -font {%s %d bold} -fill #%6.6x -text {%s} \n",
./iem/iemgui/src/sym_dial.c:  sys_vgui(".x%lx.c itemconfigure %lxSYMBOL -font {%s %d bold} -fill #%6.6x \n",

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=3518344&group_id=55736