Minor change wrt #747

Author: cowtowncoder

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

@@ -330,7 +330,7 @@ public enum Feature {
           * doubles or if we use {@link com.fasterxml.jackson.core.io.doubleparser}
           * instead.
           *<p>
-          * This setting is disabled by default.
+          * This setting is disabled by default for backwards compatibility.
           *
           * @since 2.14
           */

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

@@ -91,6 +91,17 @@ public enum StreamReadFeature
      */
     INCLUDE_SOURCE_IN_LOCATION(JsonParser.Feature.INCLUDE_SOURCE_IN_LOCATION),
 
+    /**
+     * Feature that determines whether we use the built-in {@link Double#parseDouble(String)} code to parse
+     * doubles or if we use {@link com.fasterxml.jackson.core.io.doubleparser}
+     * instead.
+     *<p>
+     * This setting is disabled by default.
+     *
+     * @since 2.14
+     */
+    USE_FAST_DOUBLE_PARSER(JsonParser.Feature.USE_FAST_DOUBLE_PARSER)
+
     ;
 
     /**

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

@@ -182,17 +182,6 @@ public enum JsonReadFeature
      */
     @SuppressWarnings("deprecation")
     ALLOW_TRAILING_COMMA(false, JsonParser.Feature.ALLOW_TRAILING_COMMA),
-
-    /**
-     * Feature that determines whether we use the built-in {@link Double#parseDouble(String)} code to parse
-     * doubles or if we use {@link com.fasterxml.jackson.core.io.doubleparser}
-     * instead.
-     *<p>
-     * This setting is disabled by default.
-     *
-     * @since 2.14
-     */
-    USE_FAST_DOUBLE_PARSER(false, JsonParser.Feature.USE_FAST_DOUBLE_PARSER)
     ;
 
     final private boolean _defaultState;

src/test/java/com/fasterxml/jackson/core/read/FastParserNonStandardNumberParsingTest.java

@@ -1,13 +1,16 @@
 package com.fasterxml.jackson.core.read;
 
 import com.fasterxml.jackson.core.JsonFactory;
+import com.fasterxml.jackson.core.StreamReadFeature;
 import com.fasterxml.jackson.core.json.JsonReadFeature;
 
-public class FastParserNonStandardNumberParsingTest extends NonStandardNumberParsingTest {
+public class FastParserNonStandardNumberParsingTest
+    extends NonStandardNumberParsingTest
+{
     private final JsonFactory fastFactory =
             JsonFactory.builder()
                     .enable(JsonReadFeature.ALLOW_LEADING_DECIMAL_POINT_FOR_NUMBERS)
-                    .enable(JsonReadFeature.USE_FAST_DOUBLE_PARSER)
+                    .enable(StreamReadFeature.USE_FAST_DOUBLE_PARSER)
                     .build();
 
     @Override

src/test/java/com/fasterxml/jackson/core/read/FastParserNumberParsingTest.java

@@ -1,11 +1,13 @@
 package com.fasterxml.jackson.core.read;
 
 import com.fasterxml.jackson.core.JsonFactory;
-import com.fasterxml.jackson.core.JsonParser;
+import com.fasterxml.jackson.core.StreamReadFeature;
 
-public class FastParserNumberParsingTest extends NumberParsingTest {
-
-    private final JsonFactory fastFactory = new JsonFactory().enable(JsonParser.Feature.USE_FAST_DOUBLE_PARSER);
+public class FastParserNumberParsingTest extends NumberParsingTest
+{
+    private final JsonFactory fastFactory = JsonFactory.builder()
+            .enable(StreamReadFeature.USE_FAST_DOUBLE_PARSER)
+            .build();
 
     @Override
     protected JsonFactory jsonFactory() {

src/test/java/com/fasterxml/jackson/core/read/NumberParsingTest.java

@@ -20,7 +20,6 @@
 public class NumberParsingTest
     extends com.fasterxml.jackson.core.BaseTest
 {
-
     protected JsonFactory jsonFactory() {
         return sharedStreamFactory();
     }