RFR: 8337225: Demote maxStack and maxLocals from CodeModel to CodeAttribute [v4]

[Chen Liang]

> 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 <turbanoff at 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