How about if (!os.startsWith("Solaris") || !os.startsWith("Linux")) { mode = -1; }
Thanks Max On 6/24/13 10:35 AM, Xuelei Fan wrote: > On 6/24/2013 10:01 AM, Weijun Wang wrote: >> >> >> On 6/24/13 9:49 AM, Xuelei Fan wrote: >>> ReplayCacheTestProc.java >>> ======================== >>> 75 String os = System.getProperty("os.name", "???"); >>> >>> What happens if "os" is "???"? >> >> Then it's neither Windows nor Mac OS X. :) >> > The code means except Windows and Mac OS X, all other platform supports > native mode, even the OS name is unknown. > >> I gave a default value so that the result will not be null. Although the >> system property "os.name" should always have a value, it makes me >> comfort not thinking of possible NPE. >> > ;-) I guess you may also worry that if the OS does not support native > mode other than Windows and Mac OS X, the test will fail again. > > As you are here already, I would suggest to make more improvement. When > the OS is unknown, java mode should be used . > > Xuelei > >> Thanks >> Max >> >>> >>> Xuelei >>> >>> On 6/24/2013 12:36 AM, Wang Weijun wrote: >>>> Send again. And BTW, JPRT runs fine. >>>> >>>> 在 Jun 23, 2013,6:29 PM,Weijun Wang <weijun.w...@oracle.com> 写道: >>>> >>>>> The macosx problem found, the machine's native GSS does not support >>>>> shared replaycache. >>>>> >>>>> *Valerie* and/or *Xuelei*, can you please review the fix? >>>>> >>>>> http://cr.openjdk.java.net/~weijun/8017453/webrev.00/ >>>>> >>>>>