The test expects ClassFileReconstituter restores exactly the same bytes as 
original classbytes.
This can be wrong if the class has more than 1 method (due to method sorting in 
the VM).
MethodParametersTarget class had only 1 method (method1), but didn't have 
constructors. This caused declaration of implicit default constructor, so the 
class actually had 2 methods.
The fix converts the method to constructor to avoid default constructor 
declaration.

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

Commit messages:
 - Fixed indent
 - Converted target class method to ctor

Changes: https://git.openjdk.java.net/jdk/pull/7345/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=7345&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8281243
  Stats: 6 lines in 1 file changed: 1 ins; 0 del; 5 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7345.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7345/head:pull/7345

PR: https://git.openjdk.java.net/jdk/pull/7345

Reply via email to