RFR: 8319373: Serial: Refactor dirty cards scanning during Young GC [v8]

Kim Barrett kbarrett at openjdk.org
Sat Nov 18 05:13:30 UTC 2023


On Fri, 17 Nov 2023 22:20:30 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> My original comment was about the .hpp file.  My followup comment noted that the qualification is needed in
>> the .cpp file, because of the different scoping (in the class definition body vs outside it).  The error you are
>> mentioning is in the .cpp file.  A leading return type is not in the scope of the function.
>
> I didn't understand your request. Compiling works fine if I remove the prefix in header only. However, I think it's more consistent when the declaration in header and definition have the identical signature.

There are a number of ways in which a declaration and a separate definition may or even must differ. 
I think this is a false consistency, and confusing because the qualifier is unnecessary and eye catching.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16492#discussion_r1398086819


More information about the hotspot-gc-dev mailing list