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
Use WCHAR variant of OutputDebugString #48325
Comments
The attached patch converts the call to OutputDebugString() with a |
The alloca() function should be avoided here: the function may be called |
I agree that a static buffer should be used. I think calling it in a |
Which CE version ? Is the patch required for previous/next CE version ? |
Roumen, just and explanation on the TCHAR/WCHAR/CHAR issue under win32... In the old days, DOS/Windows was built with 8-bit characters using Now, under CE, which is an embedded OS, neither the Which CE version? All of them, since at least CE3 (CE6 is current). Why |
Actually, even _Py_NegativeRefcount() passes a statically sized buffer |
I think this question is pointless - we don't have the power to change |
My experience with windows CE ends with version about 3.1X. I couldn't "Pointless" question is for compiler flags and is not related with the OS. |
I don't think the compiler has any such flag that you might consider |
"PythonCE project use xxxA functions for CE .NET 4.20 platform." Look again. The PythonCE project adds a header and sourcefile I was thinking of going that way too, but in the end decided against it BTW: in case somebody actually wants to resurrect the win9x support, |
I couldn't find in MSDN flags for Windows CE compilers similar to the From MSDN isn't so clear what is status of CE version 6.0 - R2?. The About wince_compatibility.h/c - it is just a part of the patch So we see changes from xxx to xxxA, xxx to xxxW, xxxA to xxx (!). If this patch required for CE 5.0 ? If not why to change now for may |
Python should use the *W functions wherever possible.
Most certainly. If we call the *A function instead, it will convert to |
"If this patch required for CE 5.0?" The patch I created is required for all CEs that I know of. I have Roumen, you mentioned the way that the PythonCE project did it, which So, what this path does is to help phase out the *A functions, gaining |
May be OutputDebugStringA has to be part of "wince-port" library but if |
Is there a problem with the proposed patch? |
Thanks for the patch. Committed as r68172. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: