Minor fix wrt #17: should not add abstract methods without default impl, causes unnecessary compatibility issues

Author: cowtowncoder

src/main/java/com/fasterxml/jackson/core/JsonGenerator.java

@@ -944,7 +944,10 @@ public void writeArray(double[] array, int offset, int length) throws IOExceptio
      *
      * @since 2.9
      */
-    public abstract void writeString(Reader reader, int len) throws IOException;
+    public void writeString(Reader reader, int len) throws IOException {
+        // Let's implement this as "unsupported" to make it easier to add new parser impls
+        _reportUnsupportedOperation();
+    }
 
     /**
      * Method for outputting a String value. Depending on context

src/main/java/com/fasterxml/jackson/core/base/GeneratorBase.java

@@ -8,7 +8,6 @@
 
 import java.io.IOException;
 import java.io.InputStream;
-import java.io.Reader;
 import java.math.BigDecimal;
 
 /**
@@ -300,11 +299,13 @@ public void writeStartObject(Object forValue) throws IOException
     @Override public void writeFieldName(SerializableString name) throws IOException {
         writeFieldName(name.getValue());
     }
-    
+
     //public abstract void writeString(String text) throws IOException;
 
     //public abstract void writeString(char[] text, int offset, int len) throws IOException;
 
+    //public abstract void writeString(Reader reader, int len) throws IOException;
+
     //public abstract void writeRaw(String text) throws IOException,;
 
     //public abstract void writeRaw(char[] text, int offset, int len) throws IOException;
@@ -341,17 +342,11 @@ public int writeBinary(Base64Variant b64variant, InputStream data, int dataLengt
         return 0;
     }
 
-    @Override // since 2.9
-    public void writeString(Reader reader, int len) throws IOException {
-        // Let's implement this as "unsupported" to make it easier to add new parser impls
-        _reportUnsupportedOperation();
-    }
-
     /*
-   /**********************************************************
-   /* Public API, write methods, primitive
-   /**********************************************************
-    */
+    /**********************************************************
+    /* Public API, write methods, primitive
+    /**********************************************************
+     */
 
     // Not implemented at this level, added as placeholders