> Java changes for supporting preview mode when preview mode resources (with 
> new location flags) are available.
> 
> At the moment, this code will operate on non-preview jimage files (1.0) and 
> act as if no preview resources are available by virtue of the default value 
> for missing attributes and package flags being zero (which matches jimage 
> 1.0).
> 
> This should be reviewed on top of 
> https://github.com/openjdk/valhalla/pull/1618

David Beaumont has updated the pull request with a new target base due to a 
merge or a rebase. The incremental webrev excludes the unrelated changes 
brought in by the merge/rebase. The pull request contains 12 additional commits 
since the last revision:

 - Rolled up changes after rebase.
   
   * Removing package root flag based on feedback.
   * Changing existing package flags during writing to match altered flag 
values.
   * Feedback changes, and fixing some comments.
   * Renaming slightly confusing "testEncoder" method.
   * Fixing unit tests to use new constructor.
   * Word smithing flags definitions.
   * Add workaround until new image writing code is in
   * Clarifying flag docs for /packages/xxx case
   * Java ImageReader changes for preview mode
 - Merge branch 'jdk_8366093_cpp/squashed' into jdk_8368333_java/squashed
 - [[RESET BRANCH FOR MERGE]]
 - Removing package root flag based on feedback.
 - Changing existing package flags during writing to match altered flag values.
 - Feedback changes, and fixing some comments.
 - Test fixes and feedback changes.
   
   * Renaming slightly confusing "testEncoder" method.
   * Fixing unit tests to use new constructor.
 - Manually deleting ImageReaderFactory (it returned somehow)
 - Word smithing flags definitions.
 - Add workaround until new image writing code is in
 - ... and 2 more: https://git.openjdk.org/valhalla/compare/77039d1f...9bbc26c1

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/1619/files
  - new: https://git.openjdk.org/valhalla/pull/1619/files/2c0db3d3..9bbc26c1

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=1619&range=06
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=1619&range=05-06

  Stats: 72 lines in 2 files changed: 24 ins; 25 del; 23 mod
  Patch: https://git.openjdk.org/valhalla/pull/1619.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1619/head:pull/1619

PR: https://git.openjdk.org/valhalla/pull/1619

Reply via email to