RFR: 8243648: Windows 32bit compile error src/jdk.incubator.jpackage/windows/native/libjpackage/VersionInfo.cpp
Alexey Semenyuk
alexey.semenyuk at oracle.com
Tue Apr 28 17:48:27 UTC 2020
Looks good!
- Alexey
On 4/28/2020 2:55 AM, Baesken, Matthias wrote:
> Hello, please review this small fix for the windows 32bit build .
>
>
> Currently we run into this compile error on Windows :
> ./src/jdk.incubator.jpackage/windows/native/libjpackage/VersionInfo.cpp(123): error C2220: warning treated as error - no 'object' file generated
> ./src/jdk.incubator.jpackage/windows/native/libjpackage/VersionInfo.cpp(123): warning C4244: 'argument': conversion from 'std::streamoff' to 'size_t', possible loss of data
>
> (after 8236129 was pushed)
>
> On windows 32bit we have
>
> typedef long streamoff
>
> according to
>
> https://docs.microsoft.com/en-us/cpp/standard-library/ios-typedefs?view=vs-2019
>
> so we had issues with the size_t conversion .
>
> Bug/webrev :
>
> https://bugs.openjdk.java.net/browse/JDK-8243648
>
> http://cr.openjdk.java.net/~mbaesken/webrevs/8243648.0/
>
> Thanks, Matthias
More information about the core-libs-dev
mailing list