<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000">
                                        
                                        
                                            
                                        
                                        
                                        Hi Eliot,<div><br></div><div>I suppose we decided to make explicit use of the unicode functions. See <span style="font-size: 10pt"><a href="https://devblogs.microsoft.com/oldnewthing/?p=40643" style="font-size: 10pt">https://devblogs.microsoft.com/oldnewthing/?p=40643</a> -- Maybe this is from a time when our source held both ANSI and UNICODE variants. These days, we could simplify the code by defining -DUNICODE=1 and skip using, e.g., "GetWindowTextW" because then "GetWindowText" will automatically choose the unicode versions.</span></div><div><span style="font-size: 10pt"><br></span></div><div><span style="font-size: 10pt">I think that that -DUNICODE has never been part of some makefile.</span></div><div><span style="font-size: 10pt"><br></span></div><div><span style="font-size: 10pt">Best,</span></div><div><span style="font-size: 10pt">Marcel</span></div><div class="mb_sig"></div>
                                        
                                        <blockquote class="history_container" type="cite" style="border-left-style: solid;border-width: 1px;margin-top: 20px;margin-left: 0px;padding-left: 10px;min-width: 500px">
                        <p style="color: #AAAAAA; margin-top: 10px;">Am 07.05.2020 23:09:23 schrieb Eliot Miranda <eliot.miranda@gmail.com>:</p><div style="font-family:Arial,Helvetica,sans-serif"> Hi All, especially Windows persons,
<br>
<br>   I'm debugging the 64-bit VM in the context of Perf on win64.  I've
<br>just noticed that we don't appear to be building a UNICODE VM?!?!  I rind
<br>no define of UNICODE in the win32 platform files and no define of UNICODE
<br>on the compiler command line in the build.winXXxYY/common makefiles.  This
<br>is surely a mistake, isn't it?  If this is a regression, when and why did
<br>this occur?  I'm going to add -DUNICODE=1 to the Makefile.msvc makefiles.
<br>AFAICT this should also happen with the Cygwin Makefiles.  But before I
<br>make changes that could affect lots of people I thought I should ask here.
<br>Anyone know when and why we dropped -DUNICODE from the Cygwin build command
<br>line?
<br>
<br>_,,,^..^,,,_
<br>best, Eliot
<br><div dir="ltr"><div>Hi All, especially Windows persons,</div><div><br></div><div>   I'm debugging the 64-bit VM in the context of Perf on win64.  I've just noticed that we don't appear to be building a UNICODE VM?!?!  I rind no define of UNICODE in the win32 platform files and no define of UNICODE on the compiler command line in the build.winXXxYY/common makefiles.  This is surely a mistake, isn't it?  If this is a regression, when and why did this occur?  I'm going to add -DUNICODE=1 to the Makefile.msvc makefiles.  AFAICT this should also happen with the Cygwin Makefiles.  But before I make changes that could affect lots of people I thought I should ask here.  Anyone know when and why we dropped -DUNICODE from the Cygwin build command line?</div><br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><span style="font-size: 10pt;border-collapse: separate"><div>_,,,^..^,,,_<br></div><div>best, Eliot</div></span></div></div></div></div>
<br></div></blockquote></div>