RFR: 8254586: Replace fthrow() calls with simpler method calls in classFileParser.cpp [v2]

David Holmes dholmes at openjdk.java.net
Wed Oct 14 02:54:10 UTC 2020


On Tue, 13 Oct 2020 17:13:31 GMT, Harold Seigel <hseigel at openjdk.org> wrote:

>> Hi,
>> Please review this small change to simplify exception throwing in classFileParser.cpp  The change was tested with tiers
>> one and two on Windows, Linux x64, and Mac OSX, and with tiers 3-5 on Linux x64.
>> Thanks, Harold
>
> Harold Seigel has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8254586: Replace fthrow() calls with simpler method calls in classFileParser.cpp

Looks good - thanks for the extended cleanup. This is why we have the THROW macros to handle the return.  :)

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

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/618


More information about the hotspot-runtime-dev mailing list