The new instance already initialized these fields to null. We don't set them elsewhere unless we want them to be non-null, so we shouldn't set them here either. Signed-off-by: Shawn O. Pearce <spearce@xxxxxxxxxxx> --- .../src/org/spearce/jgit/lib/Config.java | 4 ---- 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/Config.java b/org.spearce.jgit/src/org/spearce/jgit/lib/Config.java index 4a60199..4940cc2 100644 --- a/org.spearce.jgit/src/org/spearce/jgit/lib/Config.java +++ b/org.spearce.jgit/src/org/spearce/jgit/lib/Config.java @@ -639,8 +639,6 @@ else if (values.size() == 1) { // so we must create a new section header at the end. // final Entry e = new Entry(); - e.prefix = null; - e.suffix = null; e.base = section; e.extendedBase = subsection; entries.add(e); @@ -648,8 +646,6 @@ else if (values.size() == 1) { } while (valueIndex < values.size()) { final Entry e = new Entry(); - e.prefix = null; - e.suffix = null; e.base = section; e.extendedBase = subsection; e.name = name; -- 1.6.4.rc1.186.g60aa0c -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html