work on support for JsonWriterFeature

Author: cowtowncoder

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

@@ -3,6 +3,7 @@
 import com.fasterxml.jackson.core.io.CharacterEscapes;
 import com.fasterxml.jackson.core.io.SerializedString;
 import com.fasterxml.jackson.core.json.JsonReadFeature;
+import com.fasterxml.jackson.core.json.JsonWriteFeature;
 
 /**
  * {@link com.fasterxml.jackson.core.TSFBuilder}
@@ -77,6 +78,51 @@ public JsonFactoryBuilder configure(JsonReadFeature f, boolean state) {
         return state ? enable(f) : disable(f);
     }
 
+    // // // JSON-generating features
+
+    @Override
+    public JsonFactoryBuilder enable(JsonWriteFeature f) {
+        JsonGenerator.Feature old = f.mappedFeature();
+        if (old != null) {
+            enable(old);
+        }
+        return _this();
+    }
+
+    @Override
+    public JsonFactoryBuilder enable(JsonWriteFeature first, JsonWriteFeature... other) {
+        enable(first);
+        for (JsonWriteFeature f : other) {
+            enable(f);
+        }
+        return _this();
+    }
+
+    @Override
+    public JsonFactoryBuilder disable(JsonWriteFeature f) {
+        JsonGenerator.Feature old = f.mappedFeature();
+        if (old != null) {
+            disable(old);
+        }
+        return _this();
+    }
+
+    @Override
+    public JsonFactoryBuilder disable(JsonWriteFeature first, JsonWriteFeature... other) {
+        disable(first);
+        for (JsonWriteFeature f : other) {
+            disable(f);
+        }
+        return _this();
+    }
+
+    @Override
+    public JsonFactoryBuilder configure(JsonWriteFeature f, boolean state) {
+        return state ? enable(f) : disable(f);
+    }
+    
+    // // // JSON-specific helper objects
+    
     /**
      * Method for defining custom escapes factory uses for {@link JsonGenerator}s
      * it creates.

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

@@ -86,6 +86,7 @@ public enum Feature {
          *<p>
          * Feature is enabled by default (since it is required by JSON specification).
          */
+        // @Deprecated
         QUOTE_FIELD_NAMES(true),
 
         /**
@@ -100,6 +101,7 @@ public enum Feature {
          *<p>
          * Feature is enabled by default.
          */
+        // @Deprecated
         QUOTE_NON_NUMERIC_NUMBERS(true),
 
         /**
@@ -117,6 +119,7 @@ public enum Feature {
          *<p>
          * Feature is disabled by default.
          */
+        // @Deprecated
         WRITE_NUMBERS_AS_STRINGS(false),
 
         /**
@@ -146,6 +149,7 @@ public enum Feature {
          *<p>
          * Feature is disabled by default.
          */
+        // @Deprecated
         ESCAPE_NON_ASCII(false),
 
 // 23-Nov-2015, tatu: for [core#223], if and when it gets implemented

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

@@ -3,6 +3,7 @@
 import com.fasterxml.jackson.core.io.InputDecorator;
 import com.fasterxml.jackson.core.io.OutputDecorator;
 import com.fasterxml.jackson.core.json.JsonReadFeature;
+import com.fasterxml.jackson.core.json.JsonWriteFeature;
 
 /**
  * Since 2.10, Builder class is offered for creating token stream factories
@@ -222,7 +223,29 @@ private B _failNonJSON(Object feature) {
         throw new IllegalArgumentException("Feature "+feature.getClass().getName()
                 +"#"+feature.toString()+" not supported for non-JSON backend");
     }
-    
+
+    // // // JSON-specific, writes
+
+    public B enable(JsonWriteFeature f) {
+        return _failNonJSON(f);
+    }
+
+    public B enable(JsonWriteFeature first, JsonWriteFeature... other) {
+        return _failNonJSON(first);
+    }
+
+    public B disable(JsonWriteFeature f) {
+        return _failNonJSON(f);
+    }
+
+    public B disable(JsonWriteFeature first, JsonWriteFeature... other) {
+        return _failNonJSON(first);
+    }
+
+    public B configure(JsonWriteFeature f, boolean state) {
+        return _failNonJSON(f);
+    }
+
     // // // Other configuration
 
     public B inputDecorator(InputDecorator dec) {

src/main/java/com/fasterxml/jackson/core/json/JsonWriteFeature.java

@@ -1,7 +1,5 @@
 package com.fasterxml.jackson.core.json;
 
-import java.math.BigDecimal;
-
 import com.fasterxml.jackson.core.*;
 
 /**
@@ -58,16 +56,6 @@ public enum JsonWriteFeature
      */
     WRITE_NUMBERS_AS_STRINGS(false, JsonGenerator.Feature.WRITE_NUMBERS_AS_STRINGS),
 
-    /**
-     * Feature that determines whether {@link java.math.BigDecimal} entries are
-     * serialized using {@link java.math.BigDecimal#toPlainString()} to prevent
-     * values to be written using scientific notation.
-     *<p>
-     * Feature is disabled by default, so default output mode is used; this generally
-     * depends on how {@link BigDecimal} has been created.
-     */
-    WRITE_BIGDECIMAL_AS_PLAIN(false, JsonGenerator.Feature.WRITE_BIGDECIMAL_AS_PLAIN),
-    
     /**
      * Feature that specifies that all characters beyond 7-bit ASCII
      * range (i.e. code points of 128 and above) need to be output