RFR(xs): 8155639: Remove STEP numbers from error reporting

Christian Thalinger christian.thalinger at oracle.com
Mon May 2 20:09:51 UTC 2016


Could we also move the parenthesis into the STEP macro?

-  STEP(20, "(test secondary crash 1)")
+  STEP("test secondary crash 1")

> On Apr 28, 2016, at 3:48 AM, Thomas Stüfe <thomas.stuefe at gmail.com> wrote:
> 
> Hi,
> 
> may I have reviews and a sponsor for the following small improvement?
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8155639
> webrev:
> http://cr.openjdk.java.net/~stuefe/webrevs/8155639-STEP-number-removal/webrev.00/webrev/
> 
> Basically, this change proposes to get rid of the hand-coded STEP numbers
> in error reporting in favour of using the __LINE__ macro.
> 
> The hand-coded step numbers are a constant source or errors, as they need
> to maintained in monotonous order and if they are not, error reporting
> steps are just quietly swallowed.
> 
> Using the __LINE__ macro would be self-maintaining and thus much more
> robust.
> 
> Thank you!
> 
> Thomas



More information about the hotspot-runtime-dev mailing list