[lworld] RFR: 8272306: [lworld] Fix incorrectly merged code in classListParser.cpp

Ioi Lam iklam at openjdk.java.net
Mon Aug 23 21:11:59 UTC 2021


On Mon, 23 Aug 2021 12:18:10 GMT, David Simms <dsimms at openjdk.org> wrote:

>> Please review this simple fix (and small clean up) for incorrectly merged code in classListParser.cpp.
>> 
>> This following shows the difference between valhalla and the JDK mainline after this PR:
>> 
>> 
>> $ wget https://raw.githubusercontent.com/iklam/valhalla/4da52eaf53e531e96e1e6eac460d6209916d6f2f/src/hotspot/share/cds/classListParser.cpp
>> $ wget https://raw.githubusercontent.com/iklam/valhalla/a977e73076022958f9da874d029b1bfdba92f7af/src/hotspot/share/cds/classListParser.cpp
>> $ diff classListParser.cpp classListParser.cpp.1
>
> Thanks Ioi, this would be my mess you cleaned up here, much appreciated. Changes look good

@MrSimms thanks for the review.

-------------

PR: https://git.openjdk.java.net/valhalla/pull/538


More information about the valhalla-dev mailing list