Thanks Lance, Paul,

I will refactor the code, update in place:

And I am retesting.

Thanks, Roger

On 4/6/2018 12:15 PM, Paul Sandoz wrote:

On Apr 6, 2018, at 6:50 AM, Roger Riggs <> wrote:

Please review an intermittent test bug/cleanup improvement that places temporary
files in the directory that is auto-cleaned by jtreg.


Looks ok, while you are there, and up to you, you might consider a little 
refactor of the if/else so there is no repetition of the temp file creation 
code since it’s the same on all but windows e.g:

   if (osName.startsWith("Windows")) {
     return new WindowsTest();

   File userDir = new File(System.getProperty("user.dir", "."));
   File tmpFile = File.createTempFile("ProcessTrap-", ".sh”, userDir));
   if (osName.startsWith("Linux") == true) {
     return return new UnixTest(tempFile);
   } else if …
   return null;


Thanks, Roger

Reply via email to