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

8266459: Implement JEP 411: Deprecate the Security Manager for Removal #4073

Closed
wants to merge 14 commits into from
Closed
@@ -56,7 +56,7 @@ public static void main(String[] args) throws Exception {
ProcessResults processResults =
ProcessCommunicator
.executeChildProcess(Consumer.class, new String[0]);
if (!"Hello".equals(processResults.getStdErr())) {
if (!"Hello".equals(processResults.getStdOut())) {
Copy link
Contributor

@prrace prrace May 19, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Who or what prompted this change ?

Copy link
Contributor Author

@wangweij wangweij May 19, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The child process is started with -Djava.security.manager=allow (after the other PR) and a warning will be printed to stderr. Therefore I move the message to stdout.

throw new RuntimeException("transfer of remote object failed");
}
System.out.println("ok");
@@ -69,7 +69,7 @@ public static void main(String[] args) throws Exception {
DataFlavor dataFlavor = new DataFlavor(DataFlavor.javaRemoteObjectMimeType +
";class=Hello" );
Object data = clipboard.getData(dataFlavor);
System.err.print(((Hello) data).sayHello());
System.out.print(((Hello) data).sayHello());
}

}