Skip to content
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

Disable warning C4819. #326

Conversation

msgfx
Copy link
Contributor

@msgfx msgfx commented Oct 10, 2022

When Windows locale is Japanese, About these header files, Visual C++ treats C4819 as compile error. Because, when it happens that Visual C++ compiles UTF-8 files without BOM as cp932 encoding on Japanese locale.

These header files are external, it modified is difficult, and future, more files may be.

Files:
1.Falcor\external\packman\nanovdb\include\nanovdb\NanoVDB.h
2.Falcor\external\packman\deps\include\FreeImage.h

Error Messages (include Japanese):
Falcor\external\packman\nanovdb\include\nanovdb/NanoVDB.h(1,1): error C2220: 次の警告はエラーとして処理されます Falcor\external\packman\nanovdb\include\nanovdb/NanoVDB.h(1,1): warning C4819: ファイルは、現在のコード ページ (932) で表示できない文字を含んでいます。データの損失を防ぐために、ファイルを Unicode 形式で保存してください。

Falcor\external\packman\deps\include\FreeImage.h(1,1): error C2220: 次の警告はエラーとして処理されます
Falcor\external\packman\deps\include\FreeImage.h(1,1): warning C4819: ファイルは、現在のコード ページ (932) で表示できない文字を含んでいます。データの損失を防ぐために、ファイルを Unicode 形式で保存してください。

Every time Falcor has added a version, I have added C4819.
Probably all users in Japan have done so.

So, I make pull request that it disabled warning C4819 on VC Projects.

Best regards.

Because, when it happens that Visual C++ compiles UTF-8 files without BOM as cp932 encoding on Japanese locale.
Falcor\external\packman\nanovdb\include\nanovdb\NanoVDB.h
Falcor\external\packman\deps\include\FreeImage.h

Error Messages:
Falcor\external\packman\nanovdb\include\nanovdb/NanoVDB.h(1,1): error C2220: 次の警告はエラーとして処理されます
Falcor\external\packman\nanovdb\include\nanovdb/NanoVDB.h(1,1): warning C4819: ファイルは、現在のコード ページ (932) で表示できない文字を含んでいます。データの損失を防ぐために、ファイルを Unicode 形式で保存してください。

Falcor\external\packman\deps\include\FreeImage.h(1,1): error C2220: 次の警告はエラーとして処理されます
Falcor\external\packman\deps\include\FreeImage.h(1,1): warning C4819: ファイルは、現在のコード ページ (932) で表示できない文字を含んでいます。データの損失を防ぐために、ファイルを Unicode 形式で保存してください。
@skallweitNV
Copy link
Collaborator

Thanks for the PR.

@skallweitNV skallweitNV merged commit e026e54 into NVIDIAGameWorks:master Oct 12, 2022
sugita-shumpei pushed a commit to sugita-shumpei/Falcor that referenced this pull request Oct 13, 2023
Because, when it happens that Visual C++ compiles UTF-8 files without BOM as cp932 encoding on Japanese locale.
Falcor\external\packman\nanovdb\include\nanovdb\NanoVDB.h
Falcor\external\packman\deps\include\FreeImage.h

Error Messages:
Falcor\external\packman\nanovdb\include\nanovdb/NanoVDB.h(1,1): error C2220: 次の警告はエラーとして処理されます
Falcor\external\packman\nanovdb\include\nanovdb/NanoVDB.h(1,1): warning C4819: ファイルは、現在のコード ページ (932) で表示できない文字を含んでいます。データの損失を防ぐために、ファイルを Unicode 形式で保存してください。

Falcor\external\packman\deps\include\FreeImage.h(1,1): error C2220: 次の警告はエラーとして処理されます
Falcor\external\packman\deps\include\FreeImage.h(1,1): warning C4819: ファイルは、現在のコード ページ (932) で表示できない文字を含んでいます。データの損失を防ぐために、ファイルを Unicode 形式で保存してください。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants