Currently we switch off the unused-variable warning in BUILD_LIBJLI; this 
should be changed.

We might also consider getting rid of the unused-function warning which is 
switched off in BUILD_LIBJLI :


/jdk/src/java.base/unix/native/libjli/java_md.h:58:17: error: 'GetJVMPath' 
declared 'static' but never defined [-Werror=unused-function]
   58 | static jboolean GetJVMPath(const char *jdkroot, const char *jvmtype,
      | ^~~~~~~~~~
/jdk/src/java.base/unix/native/libjli/java_md.h:60:17: error: 
'GetJDKInstallRoot' declared 'static' but never defined 
[-Werror=unused-function]
   60 | static jboolean GetJDKInstallRoot(char *path, jint pathsize, jboolean 
speculative);
      | ^~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors


(e.g. by moving the static function declarations to the c files)

---------
- [x] I confirm that I make this contribution in accordance with the [OpenJDK 
Interim AI Policy](https://openjdk.org/legal/ai).

-------------

Commit messages:
 - JDK-8391912

Changes: https://git.openjdk.org/jdk/pull/32737/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=32737&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8391912
  Stats: 6 lines in 3 files changed: 1 ins; 3 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/32737.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/32737/head:pull/32737

PR: https://git.openjdk.org/jdk/pull/32737

Reply via email to