-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
8301820: C4819 warnings were reported on Windows #12427
Conversation
👋 Welcome back ysuenaga! A progress list of the required criteria for merging this PR into |
Out of curiosity, which version of VS 2022 is this? |
I uses 17.3.6 (VS Community) on Windows 11 Pro (Japanese locale (CP932)) I saw following message on the console (It is Japanese, sorry...)
In case of stubGenerator_x86_64_poly.cpp , |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry Yasumasa, but I'd much prefer to see this split into three sub-tasks (or issues) just so we can keep the issues more clearly separated.
IIUC for stubGenerator_x86_64_poly.cpp we have a multiplication symbol instead of an ascii 'x' (likely pasted from a word processing document).
For elfFile.hpp there are smart-quotes instead of ascii "
Thanks.
@dholmes-ora I splitted this in three sub tasks as #12435, #12436, #12437. So I close this PR (but I do not close this issue on JBS until all subtasks are resolved). |
C4819 warnings were reported when I tried to build JDK on Windows with VS2022.
This PR contains changes both HotSpot and client libraries. Let me know if they should be separated.
I added C4819 to
DISABLED_WARNINGS_microsoft
for libfontmanager and for libfreetype because they are 3rd-party libraries.Progress
Issue
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk pull/12427/head:pull/12427
$ git checkout pull/12427
Update a local copy of the PR:
$ git checkout pull/12427
$ git pull https://git.openjdk.org/jdk pull/12427/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 12427
View PR using the GUI difftool:
$ git pr show -t 12427
Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk/pull/12427.diff