RFR 8039080: "jinfo server_id at host" fails with "Invalid process identifier"

[shanliang]

Jaroslav,

Is it necessary to add "ValidationException"?

Could we change the constructor JInfo to:
    private static boolean validateArgs(String[] args);
the method returns false if the args are illegal, or throw an 
IllegalArgumentException

and declare the variables "args" and "useSA" as static too,

Shanliang

Jaroslav Bachorik wrote:
> Hi,
>
> Sorry for the noise but I need to get the fix re-reviewed.
> Due to the way jtreg cooperates with TestNG when runnning in agentvm I 
> can not use package private methods or constructor or fields.
>
> The updated patch - 
> http://cr.openjdk.java.net/~jbachorik/8039080/webrev.01 - makes the 
> JInfo constructor a private one and removes the package private 
> getters. The test is using reflection to create new instances of JInfo 
> and to assert its state.
>
> Thanks,
>
> -JB-