Fix compatibilty breakage for SystemReader

Introducing a new abstract method is not nice when one
expects other to subclass them. Create default implementations
so old code that implements SystemReader does not break.
The default methods just delegate to the JVM.

Change-Id: I42cdfdcb6b29f7203697a23833dca85185b0b9b3
Signed-off-by: Robin Rosenberg <robin.rosenberg@dewire.com>

1 files changed, 8 insertions, 11 deletions

diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/util/SystemReader.java b/org.eclipse.jgit/src/org/eclipse/jgit/util/SystemReader.java
index 595691573c..3afd9e5675 100644
--- a/org.eclipse.jgit/src/org/eclipse/jgit/util/SystemReader.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/util/SystemReader.java
@@ -121,15 +121,6 @@ public abstract class SystemReader {
 		public int getTimezone(long when) {
 			return getTimeZone().getOffset(when) / (60 * 1000);
 		}
-
-		public TimeZone getTimeZone() {
-			return TimeZone.getDefault();
-		}
-
-		@Override
-		public Locale getLocale() {
-			return Locale.getDefault();
-		}
 	};
 
 	/** @return the live instance to read system properties. */
@@ -200,11 +191,17 @@ public abstract class SystemReader {
 
 	/**
 	 * @return system time zone, possibly mocked for testing
+	 * @since 1.2
 	 */
-	public abstract TimeZone getTimeZone();
+	public TimeZone getTimeZone() {
+		return TimeZone.getDefault();
+	}
 
 	/**
 	 * @return the locale to use
+	 * @since 1.2
 	 */
-	public abstract Locale getLocale();
+	public Locale getLocale() {
+		return Locale.getDefault();
+	}
 }