]> source.dussan.org Git - jgit.git/commit
[performance] Remove synthetic access$ methods in lib, util and dircache 72/59172/1
authorAndrey Loskutov <loskutov@gmx.de>
Wed, 28 Oct 2015 19:52:43 +0000 (20:52 +0100)
committerAndrey Loskutov <loskutov@gmx.de>
Wed, 28 Oct 2015 19:52:43 +0000 (20:52 +0100)
commit260e092e7966e15a8778af1d5e6bec89689f7377
tree6f76cc160aa516df6103de2ecd8162cac6dd97ea
parentdf876d2e0f4a3d4b99249ae03f901adcb9175c36
[performance] Remove synthetic access$ methods in lib, util and dircache

Java compiler must generate synthetic access methods for private methods
and fields of the enclosing class if they are accessed from inner
classes and vice versa.

While invisible in the code, those synthetic access methods exist in the
bytecode and seem to produce some extra execution overhead at runtime
(compared with the direct access to this fields or methods), see
https://git.eclipse.org/r/58948/.

By removing the "private" access modifier from affected methods and
fields we help compiler to avoid generation of synthetic access methods
and hope to improve execution performance.

To validate changes, one can either use javap or use Bytecode Outline
plugin in Eclipse. In both cases one should look for "synthetic
access$<number>" methods at the end of the class and inner class files
in question - there should be none.

NB: don't mix this "synthetic access$" methods up with "public synthetic
bridge" methods generated to allow generic method override return types.

Change-Id: Ie7b65f251ec4452d5a5ed48aa0f272cf49a9aecd
Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
org.eclipse.jgit/src/org/eclipse/jgit/dircache/DirCacheTree.java
org.eclipse.jgit/src/org/eclipse/jgit/lib/ObjectIdOwnerMap.java
org.eclipse.jgit/src/org/eclipse/jgit/lib/ObjectIdSubclassMap.java
org.eclipse.jgit/src/org/eclipse/jgit/util/BlockList.java
org.eclipse.jgit/src/org/eclipse/jgit/util/FS.java
org.eclipse.jgit/src/org/eclipse/jgit/util/RefList.java
org.eclipse.jgit/src/org/eclipse/jgit/util/RefMap.java
org.eclipse.jgit/src/org/eclipse/jgit/util/TemporaryBuffer.java