Define constants for pack config option keys

Change-Id: Ifb8227cb62370029d6774f2a22b15d6478c713ca
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
Signed-off-by: David Pursehouse <david.pursehouse@gmail.com>

diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/lib/ConfigConstants.java b/org.eclipse.jgit/src/org/eclipse/jgit/lib/ConfigConstants.java
index e607edc2eac57aa99ffbd5a847b340130d6e61c7..eef822fa4b3bde6e063b53ff7665e948f95888fb 100644 (file)
--- a/org.eclipse.jgit/src/org/eclipse/jgit/lib/ConfigConstants.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/lib/ConfigConstants.java
@@ -542,4 +542,124 @@ public final class ConfigConstants {
         * @since 5.1.13
         */
        public static final String CONFIG_JMX_SECTION = "jmx";
+
+       /**
+        * The "pack.bigfilethreshold" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_BIGFILE_THRESHOLD = "bigfilethreshold";
+
+       /**
+        * The "pack.bitmapContiguousCommitCount" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_BITMAP_CONTIGUOUS_COMMIT_COUNT = "bitmapcontiguouscommitcount";
+
+       /**
+        * The "pack.bitmapDistantCommitSpan" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_BITMAP_DISTANT_COMMIT_SPAN = "bitmapdistantcommitspan";
+
+       /**
+        * The "pack.bitmapExcessiveBranchCount" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_BITMAP_EXCESSIVE_BRANCH_COUNT = "bitmapexcessivebranchcount";
+
+       /**
+        * The "pack.bitmapInactiveBranchAgeInDays" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_BITMAP_INACTIVE_BRANCH_AGE_INDAYS = "bitmapinactivebranchageindays";
+
+       /**
+        * The "pack.bitmapRecentCommitSpan" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_BITMAP_RECENT_COMMIT_COUNT = "bitmaprecentcommitspan";
+
+       /**
+        * The "pack.buildBitmaps" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_BUILD_BITMAPS = "buildbitmaps";
+
+       /**
+        * The "pack.cutDeltaChains" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_CUT_DELTACHAINS = "cutdeltachains";
+
+       /**
+        * The "pack.deltaCacheLimit" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_DELTA_CACHE_LIMIT = "deltacachelimit";
+
+       /**
+        * The "pack.deltaCacheSize" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_DELTA_CACHE_SIZE = "deltacachesize";
+
+       /**
+        * The "pack.deltaCompression" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_DELTA_COMPRESSION = "deltacompression";
+
+       /**
+        * The "pack.depth" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_DEPTH = "depth";
+
+       /**
+        * The "pack.minSizePreventRacyPack" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_MIN_SIZE_PREVENT_RACYPACK = "minsizepreventracypack";
+
+       /**
+        * The "pack.reuseDeltas" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_REUSE_DELTAS = "reusedeltas";
+
+       /**
+        * The "pack.reuseObjects" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_REUSE_OBJECTS = "reuseobjects";
+
+       /**
+        * The "pack.singlePack" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_SINGLE_PACK = "singlepack";
+
+       /**
+        * The "pack.threads" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_THREADS = "threads";
+
+       /**
+        * The "pack.waitPreventRacyPack" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_WAIT_PREVENT_RACYPACK = "waitpreventracypack";
+
+       /**
+        * The "pack.window" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_WINDOW = "window";
+
+       /**
+        * The "pack.windowMemory" key
+        * @since 5.8
+        */
+       public static final String CONFIG_KEY_WINDOW_MEMORY = "windowmemory";
 }

diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/storage/pack/PackConfig.java b/org.eclipse.jgit/src/org/eclipse/jgit/storage/pack/PackConfig.java
index 259f01175754d1751bf5ed5b97c938fa2bb3ba0d..f76dd2721f4eae53420db0082fba2732a9a0d3b6 100644 (file)
--- a/org.eclipse.jgit/src/org/eclipse/jgit/storage/pack/PackConfig.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/storage/pack/PackConfig.java
@@ -11,6 +11,31 @@
 
 package org.eclipse.jgit.storage.pack;
 
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_CORE_SECTION;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_BIGFILE_THRESHOLD;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_BITMAP_CONTIGUOUS_COMMIT_COUNT;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_BITMAP_DISTANT_COMMIT_SPAN;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_BITMAP_EXCESSIVE_BRANCH_COUNT;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_BITMAP_INACTIVE_BRANCH_AGE_INDAYS;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_BITMAP_RECENT_COMMIT_COUNT;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_BUILD_BITMAPS;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_COMPRESSION;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_CUT_DELTACHAINS;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_DELTA_CACHE_LIMIT;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_DELTA_CACHE_SIZE;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_DELTA_COMPRESSION;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_DEPTH;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_INDEXVERSION;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_MIN_SIZE_PREVENT_RACYPACK;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_REUSE_DELTAS;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_REUSE_OBJECTS;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_SINGLE_PACK;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_THREADS;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_WAIT_PREVENT_RACYPACK;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_WINDOW;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_KEY_WINDOW_MEMORY;
+import static org.eclipse.jgit.lib.ConfigConstants.CONFIG_PACK_SECTION;
+
 import java.util.concurrent.Executor;
 import java.util.zip.Deflater;
 
@@ -1101,52 +1126,63 @@ public class PackConfig {
         *            configuration to read properties from.
         */
        public void fromConfig(Config rc) {
-               setMaxDeltaDepth(rc.getInt("pack", "depth", getMaxDeltaDepth())); //$NON-NLS-1$ //$NON-NLS-2$
-               setDeltaSearchWindowSize(rc.getInt(
-                               "pack", "window", getDeltaSearchWindowSize())); //$NON-NLS-1$ //$NON-NLS-2$
-               setDeltaSearchMemoryLimit(rc.getLong(
-                               "pack", "windowmemory", getDeltaSearchMemoryLimit())); //$NON-NLS-1$ //$NON-NLS-2$
-               setDeltaCacheSize(rc.getLong(
-                               "pack", "deltacachesize", getDeltaCacheSize())); //$NON-NLS-1$ //$NON-NLS-2$
-               setDeltaCacheLimit(rc.getInt(
-                               "pack", "deltacachelimit", getDeltaCacheLimit())); //$NON-NLS-1$ //$NON-NLS-2$
-               setCompressionLevel(rc.getInt("pack", "compression", //$NON-NLS-1$ //$NON-NLS-2$
-                               rc.getInt("core", "compression", getCompressionLevel()))); //$NON-NLS-1$ //$NON-NLS-2$
-               setIndexVersion(rc.getInt("pack", "indexversion", getIndexVersion())); //$NON-NLS-1$ //$NON-NLS-2$
-               setBigFileThreshold(rc.getInt(
-                               "core", "bigfilethreshold", getBigFileThreshold())); //$NON-NLS-1$ //$NON-NLS-2$
-               setThreads(rc.getInt("pack", "threads", getThreads())); //$NON-NLS-1$ //$NON-NLS-2$
+               setMaxDeltaDepth(rc.getInt(CONFIG_PACK_SECTION, CONFIG_KEY_DEPTH,
+                               getMaxDeltaDepth()));
+               setDeltaSearchWindowSize(rc.getInt(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_WINDOW, getDeltaSearchWindowSize()));
+               setDeltaSearchMemoryLimit(rc.getLong(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_WINDOW_MEMORY, getDeltaSearchMemoryLimit()));
+               setDeltaCacheSize(rc.getLong(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_DELTA_CACHE_SIZE, getDeltaCacheSize()));
+               setDeltaCacheLimit(rc.getInt(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_DELTA_CACHE_LIMIT, getDeltaCacheLimit()));
+               setCompressionLevel(rc.getInt(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_COMPRESSION, rc.getInt(CONFIG_CORE_SECTION,
+                                               CONFIG_KEY_COMPRESSION, getCompressionLevel())));
+               setIndexVersion(rc.getInt(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_INDEXVERSION,
+                               getIndexVersion()));
+               setBigFileThreshold(rc.getInt(CONFIG_CORE_SECTION,
+                               CONFIG_KEY_BIGFILE_THRESHOLD, getBigFileThreshold()));
+               setThreads(rc.getInt(CONFIG_PACK_SECTION, CONFIG_KEY_THREADS,
+                               getThreads()));
 
                // These variables aren't standardized
-               //
-               setReuseDeltas(rc.getBoolean("pack", "reusedeltas", isReuseDeltas())); //$NON-NLS-1$ //$NON-NLS-2$
-               setReuseObjects(
-                               rc.getBoolean("pack", "reuseobjects", isReuseObjects())); //$NON-NLS-1$ //$NON-NLS-2$
-               setDeltaCompress(
-                               rc.getBoolean("pack", "deltacompression", isDeltaCompress())); //$NON-NLS-1$ //$NON-NLS-2$
-               setCutDeltaChains(
-                               rc.getBoolean("pack", "cutdeltachains", getCutDeltaChains())); //$NON-NLS-1$ //$NON-NLS-2$
-               setSinglePack(
-                               rc.getBoolean("pack", "singlepack", getSinglePack())); //$NON-NLS-1$ //$NON-NLS-2$
-               setBuildBitmaps(
-                               rc.getBoolean("pack", "buildbitmaps", isBuildBitmaps())); //$NON-NLS-1$ //$NON-NLS-2$
-               setBitmapContiguousCommitCount(
-                               rc.getInt("pack", "bitmapcontiguouscommitcount", //$NON-NLS-1$ //$NON-NLS-2$
-                                               getBitmapContiguousCommitCount()));
-               setBitmapRecentCommitCount(rc.getInt("pack", "bitmaprecentcommitcount", //$NON-NLS-1$ //$NON-NLS-2$
+               setReuseDeltas(rc.getBoolean(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_REUSE_DELTAS, isReuseDeltas()));
+               setReuseObjects(rc.getBoolean(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_REUSE_OBJECTS, isReuseObjects()));
+               setDeltaCompress(rc.getBoolean(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_DELTA_COMPRESSION, isDeltaCompress()));
+               setCutDeltaChains(rc.getBoolean(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_CUT_DELTACHAINS, getCutDeltaChains()));
+               setSinglePack(rc.getBoolean(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_SINGLE_PACK,
+                               getSinglePack()));
+               setBuildBitmaps(rc.getBoolean(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_BUILD_BITMAPS, isBuildBitmaps()));
+               setBitmapContiguousCommitCount(rc.getInt(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_BITMAP_CONTIGUOUS_COMMIT_COUNT,
+                               getBitmapContiguousCommitCount()));
+               setBitmapRecentCommitCount(rc.getInt(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_BITMAP_RECENT_COMMIT_COUNT,
                                getBitmapRecentCommitCount()));
-               setBitmapRecentCommitSpan(rc.getInt("pack", "bitmaprecentcommitspan", //$NON-NLS-1$ //$NON-NLS-2$
+               setBitmapRecentCommitSpan(rc.getInt(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_BITMAP_RECENT_COMMIT_COUNT,
                                getBitmapRecentCommitSpan()));
-               setBitmapDistantCommitSpan(rc.getInt("pack", "bitmapdistantcommitspan", //$NON-NLS-1$ //$NON-NLS-2$
+               setBitmapDistantCommitSpan(rc.getInt(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_BITMAP_DISTANT_COMMIT_SPAN,
                                getBitmapDistantCommitSpan()));
-               setBitmapExcessiveBranchCount(rc.getInt("pack", //$NON-NLS-1$
-                               "bitmapexcessivebranchcount", getBitmapExcessiveBranchCount())); //$NON-NLS-1$
-               setBitmapInactiveBranchAgeInDays(
-                               rc.getInt("pack", "bitmapinactivebranchageindays", //$NON-NLS-1$ //$NON-NLS-2$
-                                               getBitmapInactiveBranchAgeInDays()));
-               setWaitPreventRacyPack(rc.getBoolean("pack", "waitpreventracypack", //$NON-NLS-1$ //$NON-NLS-2$
-                               isWaitPreventRacyPack()));
-               setMinSizePreventRacyPack(rc.getLong("pack", "minsizepreventracypack", //$NON-NLS-1$//$NON-NLS-2$
+               setBitmapExcessiveBranchCount(rc.getInt(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_BITMAP_EXCESSIVE_BRANCH_COUNT,
+                               getBitmapExcessiveBranchCount()));
+               setBitmapInactiveBranchAgeInDays(rc.getInt(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_BITMAP_INACTIVE_BRANCH_AGE_INDAYS,
+                               getBitmapInactiveBranchAgeInDays()));
+               setWaitPreventRacyPack(rc.getBoolean(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_WAIT_PREVENT_RACYPACK, isWaitPreventRacyPack()));
+               setMinSizePreventRacyPack(rc.getLong(CONFIG_PACK_SECTION,
+                               CONFIG_KEY_MIN_SIZE_PREVENT_RACYPACK,
                                getMinSizePreventRacyPack()));
        }