Code Review for webrev

Prepared by: Maurizio Cimadamore on Fri May 15 14:41:36 UTC 2020
Compare against: https://git.openjdk.java.net/panama-foreign
Compare against version: 39ab6186
Summary of changes: 170 lines changed; 128 ins; 38 del; 4 mod; 2523 unchg
Patch of changes: webrev.patch
Pull request: 169
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/MemoryAddress.java

b945c8f5: Move static MemoryAddress::copy into instance method inside MemorySegment  
27 lines changed; 0 ins; 27 del; 0 mod; 109 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/MemorySegment.java

b945c8f5: Move static MemoryAddress::copy into instance method inside MemorySegment  
24 lines changed; 24 ins; 0 del; 0 mod; 607 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/jdk.incubator.foreign/share/classes/jdk/internal/foreign/AbstractMemorySegmentImpl.java

b945c8f5: Move static MemoryAddress::copy into instance method inside MemorySegment  
12 lines changed; 11 ins; 0 del; 1 mod; 470 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/jdk.incubator.foreign/share/classes/jdk/internal/foreign/MemoryAddressImpl.java

b945c8f5: Move static MemoryAddress::copy into instance method inside MemorySegment  
11 lines changed; 0 ins; 11 del; 0 mod; 125 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/java/foreign/TestByteBuffer.java

b945c8f5: Move static MemoryAddress::copy into instance method inside MemorySegment  
2 lines changed; 0 ins; 0 del; 2 mod; 649 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/java/foreign/TestMemoryCopy.java

b945c8f5: Move static MemoryAddress::copy into instance method inside MemorySegment  
1 lines changed; 0 ins; 0 del; 1 mod; 122 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/java/foreign/TestSegments.java

b945c8f5: Move static MemoryAddress::copy into instance method inside MemorySegment  
1 lines changed; 1 ins; 0 del; 0 mod; 441 unchg

------ ------ ------ ------ --- New Patch Raw test/micro/org/openjdk/bench/jdk/incubator/foreign/BulkOps.java

7da26e8c: Fixed copyright
3776b9e9: Added benchmark for bulk operations  
92 lines changed; 92 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using webrev version a0340ac5221e72655f391ac20ca847038b353113