> Hello, > > We should consider moving the enum ArrayProperties from ArrayKlass to its own > class, ArrayProperties. In addition to making the code easier to read and > understand, this allows us to have explicit setters/getters, replacing the > bit-fiddling expressions that are used in many places. The > ArrayProperties-specific methods in ArrayKlass have been moved to be methods > in the new ArrayProperties class instead. > > Perhaps the most controversial change in this PR is the removal of > `ArrayKlass::ArrayProperties::DEFAULT` in favor of using a default > constructor for ArrayProperties. The semantics are still the same, i.e., > asking `.is_null_restricted()` or `.is_non_atomic()` will be false for the > default constructed property. With this I've also removed the unused fields > from ArrayProperties (DUMMY and comments). > > I did consider using define macros to generate enum+getters+setters, but I > opted for the stamped-out version instead. > > Testing: > * Oracle's tier1-4
Joel Sikström has updated the pull request incrementally with one additional commit since the last revision: Explicit bool checks ------------- Changes: - all: https://git.openjdk.org/valhalla/pull/2114/files - new: https://git.openjdk.org/valhalla/pull/2114/files/e3a87a4a..e53199c3 Webrevs: - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2114&range=06 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2114&range=05-06 Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod Patch: https://git.openjdk.org/valhalla/pull/2114.diff Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2114/head:pull/2114 PR: https://git.openjdk.org/valhalla/pull/2114
