> Please review this change that moves the Throwable and StackTrace classes 
> into a new file.  I didn't make any modifications to the code, just moved it, 
> added include directives to javaStackTraceClasses.hpp, minimized include 
> directives in the new files, and removed includes in javaClasses.cpp that 
> were no longer needed.
> 
> Tested with tier1 on Oracle supported platforms (product & fastdebug).
> 
> This change to be merged post valhalla integration.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Coleen Phillimore has updated the pull request with a new target base due to a 
merge or a rebase. The pull request now contains six commits:

 - Merge branch 'master' into throwable
 - Remove useless friend and obsolete comment.
 - This class makes no sense to be friends with JavaClasses.
 - Add forward declarations, fix MAX_VERSION that fails with product compiles
 - Add javaStackTraceClasses.cpp
 - 8388791: Refactor JavaClasses Throwable and StackTrace classes

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

Changes: https://git.openjdk.org/jdk/pull/32016/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=32016&range=04
  Stats: 3006 lines in 40 files changed: 1546 ins; 1450 del; 10 mod
  Patch: https://git.openjdk.org/jdk/pull/32016.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/32016/head:pull/32016

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

Reply via email to