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

Axel Boldt-Christmas duke at openjdk.org
Fri Aug 5 08:27:06 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.
> 
> Testing: Tier 1-3

This pull request has now been integrated.

Changeset: dd7f2d91
Author:    Axel Boldt-Christmas <axel.boldt-christmas at oracle.com>
Committer: Erik Österlund <eosterlund at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/dd7f2d912bb66c0f10d7165040e52e2d18b73897
Stats:     78 lines in 11 files changed: 7 ins; 4 del; 67 mod

8290090: Change CodeBlobType from unscoped enum to enum class

Reviewed-by: eosterlund, kvn

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

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


More information about the hotspot-dev mailing list