This constant was pointless as changing it would brake code in the same way as changing hardcoded value instead. Signed-off-by: Marek Zawirski <marek.zawirski@xxxxxxxxx> --- .../src/org/spearce/jgit/pgm/Push.java | 2 +- .../src/org/spearce/jgit/transport/Transport.java | 7 +------ 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/org.spearce.jgit.pgm/src/org/spearce/jgit/pgm/Push.java b/org.spearce.jgit.pgm/src/org/spearce/jgit/pgm/Push.java index b61071c..d36bf4f 100644 --- a/org.spearce.jgit.pgm/src/org/spearce/jgit/pgm/Push.java +++ b/org.spearce.jgit.pgm/src/org/spearce/jgit/pgm/Push.java @@ -87,7 +87,7 @@ void nothin(final boolean ignored) { private String receivePack; @Option(name = "--dry-run") - private boolean dryRun = Transport.DEFAULT_DRY_RUN; + private boolean dryRun; private boolean shownURI; diff --git a/org.spearce.jgit/src/org/spearce/jgit/transport/Transport.java b/org.spearce.jgit/src/org/spearce/jgit/transport/Transport.java index 939347e..ca68ca6 100644 --- a/org.spearce.jgit/src/org/spearce/jgit/transport/Transport.java +++ b/org.spearce.jgit/src/org/spearce/jgit/transport/Transport.java @@ -313,11 +313,6 @@ private static String findTrackingRefName(final String remoteName, public static final RefSpec REFSPEC_PUSH_ALL = new RefSpec( "refs/heads/*:refs/heads/*"); - /** - * Default setting for {@link #dryRun} option. - */ - public static final boolean DEFAULT_DRY_RUN = false; - /** The repository this transport fetches into, or pushes out of. */ protected final Repository local; @@ -354,7 +349,7 @@ private static String findTrackingRefName(final String remoteName, private boolean pushThin = DEFAULT_PUSH_THIN; /** Should push just check for operation result, not really push. */ - private boolean dryRun = DEFAULT_DRY_RUN; + private boolean dryRun; /** * Create a new transport instance. -- 1.5.6.3 -- 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