RFR(T): 8221738: ErrorFile option does not handle pre-existing error files of the same name

[coleen.phillimore at oracle.com]

I think this looks very reasonable.   OutputAnalyzer now has a 
constructor that takes a file name as a parameter.  You should use this 
instead for your test.

Thanks,
Coleen


On 4/7/19 3:17 AM, Thomas Stüfe wrote:
> Hi all,
>
> May I please have reviews for this small fix:
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8221738
> cr:
> http://cr.openjdk.java.net/~stuefe/webrevs/8221738-errorfile-option-does-not-handle-pre-existing-error-files-of-the-same-name/webrev.00/webrev/
>
> Fixes a long standing issue where -XX:ErrorFile=<somename> will only work
> if <somename> does not exist yet. If it does, error file falls silently
> back to "<curdir>/hs_err_pid...".
>
> For more detailed discussions, please see the bug and the associated CSR.
>
> The fix now causes the error file to be overwritten
>
> Thanks, Thomas