Skip to content

Commit

Permalink
fixup comments in helper class
Browse files Browse the repository at this point in the history
  • Loading branch information
kevinrushforth committed Mar 1, 2024
1 parent 691d52b commit d3bf81e
Showing 1 changed file with 6 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@
* or visit www.oracle.com if you need additional information or have any
* questions.
*/

// TODO: incubator template
package com.sun.javafx;

import java.security.AccessController;
Expand Down Expand Up @@ -56,7 +58,7 @@ public static void incubatorWarning() {

// If we are using incubating API from the unnamed module, issue
// a warning one time for each package. This is not a supported
// mode, but can happen if the module is placed on the classpath.
// mode, but can happen if the modular jar is put on the classpath.
if (!callerModule.isNamed()) {
var callerPackage = callerClass.getPackage();
if (!warnedPackages.contains(callerPackage)) {
Expand All @@ -66,10 +68,11 @@ public static void incubatorWarning() {
return null;
}

// TODO: Check whether this module is jlinked into the runtime and
// thus has already printed a warning.
// Issue warning one time for this module
if (!warnedModules.contains(callerModule)) {
// FIXME: Check whether this module is jlinked into the runtime
// and thus has already printed a warning. Skip the warning in that
// case to avoid duplicate warnings.
System.err.println("WARNING: Using incubator modules: " + callerModule.getName());
warnedModules.add(callerModule);
}
Expand Down

0 comments on commit d3bf81e

Please sign in to comment.