On Fri, 16 Dec 2011, Alex Henrie wrote:
dlls/comdlg32/filedlg.c | 2 +-
[...]
WCHAR buf[16];
WCHAR buf[17]; LoadStringW(COMDLG32_hInstance, IDS_SAVE_AS, buf, sizeof(buf)/sizeof(WCHAR));
I don't think it makes sense to add room for just one more WCHAR. There's no telling how long a future Arabic or Panjabi translation might be. So either set the buffer size to something like 80, 128 or 256 characters (sucks 5, 8 and 16 times less respectively), or do away with the hardcoded buffer size altogether (overkill? rocks? both?<g>).
Thanks for the feedback. It looks like Alexandre widened the buffer to 64 characters when he committed the patch.
-Alex