<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Mar 22, 2020, 2:46 PM Esme <<a href="mailto:vincent@codeweavers.com" rel="noreferrer noreferrer noreferrer" target="_blank">vincent@codeweavers.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Vincent Povirk <<a href="mailto:vincent@codeweavers.com" rel="noreferrer noreferrer noreferrer noreferrer" target="_blank">vincent@codeweavers.com</a>><br>
<br>
Wine-bug: <a href="https://bugs.winehq.org/show_bug.cgi?id=47872" rel="noreferrer noreferrer noreferrer noreferrer noreferrer" target="_blank">https://bugs.winehq.org/show_bug.cgi?id=47872</a><br>
Signed-off-by: Vincent Povirk <<a href="mailto:vincent@codeweavers.com" rel="noreferrer noreferrer noreferrer noreferrer" target="_blank">vincent@codeweavers.com</a>><br>
---<br>
 dlls/gdiplus/tests/font.c | 14 +++-----------<br>
 1 file changed, 3 insertions(+), 11 deletions(-)<br>
<br>
diff --git a/dlls/gdiplus/tests/font.c b/dlls/gdiplus/tests/font.c<br>
index 9abb80faaf5..7f4b1e876e5 100644<br>
--- a/dlls/gdiplus/tests/font.c<br>
+++ b/dlls/gdiplus/tests/font.c<br>
@@ -808,29 +808,21 @@ static void test_font_metrics(void)<br>
<br>
 static void test_font_substitution(void)<br>
 {<br>
-    WCHAR ms_shell_dlg[LF_FACESIZE];<br>
+    const WCHAR ms_shell_dlg[] = {'M','S',' ','S','h','e','l','l',' ','D','l','g',0};</blockquote></div></div><div dir="auto"><br></div><div dir="auto">Especially for tests, is there any reason now to avoid using L””-style literals for wchar strings?</div><div dir="auto"><br></div><div dir="auto">If not, you could get rid of ms_shell_dlg completely and substitute L"MS Shell Dlg" for the one remaining place ms_shell_dlg is used.</div></div>