-
Bug
-
Resolution: Fixed
-
P4
-
11, 13
-
b24
Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-8225947 | 14 | Philip Race | P4 | Resolved | Fixed | team |
-------- Forwarded Message --------
Subject: <Sound Dev> [PATCH] Fix some assignments of string literals to LPSTR (instead of LPCSTR)
Date: Tue, 14 May 2019 00:36:49 +0000
From: Andrew Luo <andrewluotechnologies@outlook.com>
To: sound-dev@openjdk.java.net <sound-dev@openjdk.java.net>
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,
- backported by
-
JDK-8225947 Fix some assignments of string literals to LPSTR (instead of LPCSTR)
-
- Resolved
-