-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
64-bit windows builds are broken since uncode version of windows api is called #11
Comments
Can you please check if the develop branch is working? |
<!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
.MsoChpDefault
{mso-style-type:export-only;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:2.0cm 42.5pt 2.0cm 3.0cm;}
div.WordSection1
{page:WordSection1;}
-->Johannes, It compiles. I’ve tried to log something with SinkOutputDebugString, SinkEventLog and it outputs strings to debug console/event log, so it seems to be working. Andy From: Johannes PohlSent: 21 апреля 2020 г. 11:12To: badaix/aixlogCc: andyp71; AuthorSubject: Re: [badaix/aixlog] 64-bit windows builds are broken since uncode version of windows api is called (#11) Can you please check if the develop branch is working?There happened some development in the Snapcast project which I've now merged into develop.Regarding Windows: I can just take pull requests or fix trivial and obvious things, that can be fixed without having a Windows machine.—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.
|
Cool! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
You'd better replace
OutputDebugString with OutputDebugStringA
RegisterEventSource with RegisterEventSourceA
ReportEvent with ReportEventA
You should call the "A" version of these functions explicitly, since you pass ansi char strings to them.
Otherwise 64-bit windows builds are broken. OutputDebugString is defined to OutputDebugStringW on 64-bit platform and compiler can not convert const char* to required LPCWSTR.
The text was updated successfully, but these errors were encountered: