Minor refactoring of JsonGenerator.copyCurrentEvent()/copyCurrentStructure() to be more modular

Author: cowtowncoder

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

@@ -2471,36 +2471,14 @@ public void copyCurrentEvent(JsonParser p) throws IOException
             writeFieldName(p.getCurrentName());
             break;
         case ID_STRING:
-            if (p.hasTextCharacters()) {
-                writeString(p.getTextCharacters(), p.getTextOffset(), p.getTextLength());
-            } else {
-                writeString(p.getText());
-            }
+            _copyCurrentStringValue(p);
             break;
         case ID_NUMBER_INT:
-        {
-            NumberType n = p.getNumberType();
-            if (n == NumberType.INT) {
-                writeNumber(p.getIntValue());
-            } else if (n == NumberType.BIG_INTEGER) {
-                writeNumber(p.getBigIntegerValue());
-            } else {
-                writeNumber(p.getLongValue());
-            }
+            _copyCurrentIntValue(p);
             break;
-        }
         case ID_NUMBER_FLOAT:
-        {
-            Number n = p.getNumberValueExact();
-            if (n instanceof BigDecimal) {
-                writeNumber((BigDecimal) n);
-            } else if (n instanceof Double) {
-                writeNumber(n.doubleValue());
-            } else {
-                writeNumber(n.floatValue());
-            }
+            _copyCurrentFloatValue(p);
             break;
-        }
         case ID_TRUE:
             writeBoolean(true);
             break;
@@ -2616,36 +2594,14 @@ protected void _copyCurrentContents(JsonParser p) throws IOException
                 break;
 
             case ID_STRING:
-                if (p.hasTextCharacters()) {
-                    writeString(p.getTextCharacters(), p.getTextOffset(), p.getTextLength());
-                } else {
-                    writeString(p.getText());
-                }
+                _copyCurrentStringValue(p);
                 break;
             case ID_NUMBER_INT:
-            {
-                NumberType n = p.getNumberType();
-                if (n == NumberType.INT) {
-                    writeNumber(p.getIntValue());
-                } else if (n == NumberType.BIG_INTEGER) {
-                    writeNumber(p.getBigIntegerValue());
-                } else {
-                    writeNumber(p.getLongValue());
-                }
+                _copyCurrentIntValue(p);
                 break;
-            }
             case ID_NUMBER_FLOAT:
-            {
-                Number n = p.getNumberValueExact();
-                if (n instanceof BigDecimal) {
-                    writeNumber((BigDecimal) n);
-                } else if (n instanceof Double) {
-                    writeNumber(n.doubleValue());
-                } else {
-                    writeNumber(n.floatValue());
-                }
+                _copyCurrentFloatValue(p);
                 break;
-            }
             case ID_TRUE:
                 writeBoolean(true);
                 break;
@@ -2664,6 +2620,63 @@ protected void _copyCurrentContents(JsonParser p) throws IOException
         }
     }
 
+    /**
+     * Method for copying current {@link JsonToken#VALUE_NUMBER_FLOAT} value;
+     * overridable by format backend implementations.
+     *
+     * @param p Parser that points to the value to copy
+     *
+     * @since 2.15
+     */
+    protected void _copyCurrentFloatValue(JsonParser p) throws IOException
+    {
+        Number n = p.getNumberValueExact();
+        if (n instanceof BigDecimal) {
+            writeNumber((BigDecimal) n);
+        } else if (n instanceof Double) {
+            writeNumber(n.doubleValue());
+        } else {
+            writeNumber(n.floatValue());
+        }
+    }
+
+    /**
+     * Method for copying current {@link JsonToken#VALUE_NUMBER_FLOAT} value;
+     * overridable by format backend implementations.
+     *
+     * @param p Parser that points to the value to copy
+     *
+     * @since 2.15
+     */
+    protected void _copyCurrentIntValue(JsonParser p) throws IOException
+    {
+        NumberType n = p.getNumberType();
+        if (n == NumberType.INT) {
+            writeNumber(p.getIntValue());
+        } else if (n == NumberType.LONG) {
+            writeNumber(p.getLongValue());
+        } else {
+            writeNumber(p.getBigIntegerValue());
+        }
+    }
+
+    /**
+     * Method for copying current {@link JsonToken#VALUE_STRING} value;
+     * overridable by format backend implementations.
+     *
+     * @param p Parser that points to the value to copy
+     *
+     * @since 2.15
+     */
+    protected void _copyCurrentStringValue(JsonParser p) throws IOException
+    {
+        if (p.hasTextCharacters()) {
+            writeString(p.getTextCharacters(), p.getTextOffset(), p.getTextLength());
+        } else {
+            writeString(p.getText());
+        }
+    }
+    
     /*
     /**********************************************************************
     /* Public API, buffer handling