> As discussed in offline meeting, the max stack and locals information are 
> part of the code attribute and not meaningful for buffered code elements. 
> Computation would be costly and these see no real usage during 
> transformations. Thus, the proposed solution is to move these APIs to be 
> CodeAttribute specific, as this is already how all these APIs' users are 
> using.
> 
> Also removed useless `Writable` on buffered models, and fixed 
> `BufferedMethodBuilder::code` implementation.

Chen Liang has updated the pull request incrementally with one additional 
commit since the last revision:

  Spacing style issue
  
  Co-authored-by: Andrey Turbanov <turban...@gmail.com>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/20338/files
  - new: https://git.openjdk.org/jdk/pull/20338/files/e1575ab7..8cf6b47f

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=20338&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=20338&range=02-03

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/20338.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/20338/head:pull/20338

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

Reply via email to