]> source.dussan.org Git - jgit.git/commit
Clarify the purpose of ObjectInserter.buffer() 67/5367/2
authorShawn O. Pearce <spearce@spearce.org>
Thu, 15 Mar 2012 14:57:56 +0000 (07:57 -0700)
committerChris Aniszczyk <zx@twitter.com>
Sun, 18 Mar 2012 13:20:06 +0000 (08:20 -0500)
commit04ab2dac3728f8e43dc42a9e700819dd2e7a4a93
tree02870fcaea37f90d6d1a832dfc1eb822638a143c
parentc75aa1aed238dd62d6f00958a5fb2cf782349ee1
Clarify the purpose of ObjectInserter.buffer()

Recently Robin tried to increase the size of the buffer used by
ObjectInserter to fix a bug in the InputStream that handles AutoCRLF.
The purpose of this buffer is NOT to make a random InputStream work
correctly by passing it a larger buffer during read(byte[],int,int).

Clarify the Javadoc on the buffer() method to reduce the risk
someone tries to abuse it again.

While we are here, modify the method to load the field into a local
variable before returning. This should cut down 1 field load during
the common case of the buffer being already allocated.

Change-Id: Ic6898530d10fcd7e59f90397117a4a0d97e1f031
Signed-off-by: Chris Aniszczyk <zx@twitter.com>
org.eclipse.jgit/src/org/eclipse/jgit/lib/ObjectInserter.java