RFR: 8290090: Change CodeBlobType from unscoped enum to enum class

Vladimir Kozlov kvn at openjdk.org
Mon Jul 25 17:36:07 UTC 2022


On Tue, 12 Jul 2022 07:55:49 GMT, Axel Boldt-Christmas <duke at openjdk.org> wrote:

> Change: 
> ```C++
> struct CodeBlobType { 
>   enum { [...] } 
> }
> 
> To: 
> ```C++
> enum class CodeBlobType { 
>   [...] 
> }; 
> 
> Using C++11 scoped enums provides a more clear view of intent, as enums can be enforced by the type system instead of being passed around as ints.

Looks good. I assume you tested it.

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

Marked as reviewed by kvn (Reviewer).

PR: https://git.openjdk.org/jdk/pull/9460


More information about the hotspot-dev mailing list