RFR [12] 8212001: Verify exported symbols in java.base (libjava)
Pavel Rappo
pavel.rappo at oracle.com
Thu Oct 11 12:28:07 UTC 2018
Hello,
Please review the following change:
http://cr.openjdk.java.net/~prappo/8212001/webrev.00
This is yet another clean-up kind of change. It seals unnecessarily exported
functions. This time from libjava. Also couldn't help fixing a typo. Not only
does it soothe aesthetic discomfort, it also helps with search (including
correct indexing). This change has been built and tested on all supported
platforms.
A note for reviewers. I noticed an asymmetry in the following declarations[^1]:
JNIEXPORT jint handleRead(FD fd, void *buf, jint len);
JNIEXPORT jlong JNICALL handleLseek(FD fd, jlong offset, jint whence);
These functions seem to be used in exactly the same way (from libzip). Is there
any reason for this difference (JNICALL)?
[^1]: http://cr.openjdk.java.net/~prappo/8212001/webrev.00/src/java.base/windows/native/libjava/io_util_md.h.html (lines 48, 52 and 53)
More information about the core-libs-dev
mailing list