fixed bug

TODO

@@ -13,7 +13,6 @@ To do before next release:
 - figure out a better gnustep path for wprefs.app installation
 - pick a good font fallback
 - figure a better fullscreen level
-- add default pixelsize if font doesn't define any size.
 - clean up netwm hints on exit
 - check whether window states are being saved/restored properly via netwm
   on restart/crash-restart (grep for XXX/TODO)

WPrefs.app/FontSimple.c

@@ -553,15 +553,18 @@ showData(_Panel *panel)
     
     for (i= 0; i < WMGetPopUpButtonNumberOfItems(panel->optionP); i++)
     {
-        char *ofont;
+        char *ofont, *font;
         
         item= WMGetPopUpButtonMenuItem(panel->optionP, i);
         
         ofont= WMGetMenuItemRepresentedObject(item);
         if (ofont)
           wfree(ofont);
-        
-        WMSetMenuItemRepresentedObject(item, GetStringForKey(fontOptions[i].option));
+
+        font= GetStringForKey(fontOptions[i].option);
+        if (font)
+          font= wstrdup(font);
+        WMSetMenuItemRepresentedObject(item, font);
     }
 
     WMSetPopUpButtonSelectedItem(panel->optionP, 0);
@@ -581,7 +584,7 @@ storeData(_Panel *panel)
         item= WMGetPopUpButtonMenuItem(panel->optionP, i);
 
         font= WMGetMenuItemRepresentedObject(item);
-        if (font)
+        if (font && *font)
         {
             SetStringForKey(font, fontOptions[i].option);
         }