Hi, the fix looks correct to me.
Florian
On 14.05.2019 02:36, Andrew Luo wrote:
I’ve been trying to get the JDK to compile with the new /permissive-
switch in VS2017
(https://devblogs.microsoft.com/cppblog/permissive-switch/) however
there have been a few issues in various parts of the code. There are
a few issues in the libjsound code where we attempt to assign string
literals to LPSTR variables (instead of LPCSTR). I took some time to
fix some of these and attached a patch. Let me know if you have any
comments/feedback. I’m assuming that libjsound is owned by sound-dev,
but if I’m mistaken, please do let me know the right team/mailing list
to send this to.
c:/Users/Andrew/Documents/mercurial/openjdk/jdk/src/java.desktop/windows/native/libjsound/PLATFORM_API_WinOS_DirectSound.cpp(363):
error C2440: 'return': cannot convert from 'const char [16]' to 'LPSTR'
Thanks,
-Andrew
--
Florian Bomers
Bome Software
everything sounds.
http://www.bome.com
__________________________________________________________________
Bome Software GmbH & Co KG Gesellschafterin:
Petra-Kelly-Str. 15 Bome Komplementär GmbH
80797 München, Germany Geschäftsführung: Florian Bömers
Amtsgericht München HRA95502 Amtsgericht München HRB185574
