Re: [Tuxpaint-devel] UI font configuration

[Shin-ichi T. <dol...@wm...>]

On Sun, 4 Jun 2023 03:39:33 -0700, Bill Kendrick wrote:
>Hrm, this is not happening for me.  My config, for example:

Well, this might be a specific problem (at least?) on Windows.

>What's happening on my system is when we use wordexp() to expand
>the argument, it seems to be splitting things on spaces.  From the
>manpage:

This part is skipped on WIN32 because it does not have wordexp().
Therefore, no splitting here, and the result has double quotes like.

  Requested UI font described by ""DejaVu Sans""
  Actual UI font will be "ＭＳ ゴシック"

So, should we do some silly thing like

-----------------------------------
diff --git a/src/tuxpaint-config2.cxx b/src/tuxpaint-config2.cxx
index e72acac..dc74842 100644
--- a/src/tuxpaint-config2.cxx
+++ b/src/tuxpaint-config2.cxx
@@ -782,7 +782,11 @@ void save_conf(void)
     fprintf(fd, "alllocalefonts=yes\n");

   if ((TEXTINP_uifont_isdef()) == 0)
+#ifdef WIN32
+    fprintf(fd, "uifont=%s\n", BROWSER_uifont->text(BROWSER_uifont->value()));
+#else
     fprintf(fd, "uifont=\"%s\"\n", BROWSER_uifont->text(BROWSER_uifont->value()));
+#fi

   /* Printing: */
   /* --------- */
-----------------------------------

in tuxpaint-config ?