Removed TODOs (created follow up issue helidon-io#3200)

Signed-off-by: Tomas Langer <tomas.langer@oracle.com>
tomas-langer · Jul 19, 2021 · ce903ef · ce903ef
1 parent c60d517
commit ce903ef
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 4 deletions.
diff --git a/common/common/src/main/java/io/helidon/common/SerializationConfig.java b/common/common/src/main/java/io/helidon/common/SerializationConfig.java
@@ -257,7 +257,6 @@ private void handleBadFilter(Action action, String message) {
             LOGGER.warning(message);
             break;
         case CONFIGURE:
-            // TODO this will change with Java 17, where we can create a combined filter
             throw new IllegalStateException("Cannot reconfigure current global deserialization filter."
                                                     + " Original message: " + message);
         case IGNORE:
@@ -362,9 +361,7 @@ public enum TraceOption {
      * {@link SerializationConfig#configureRuntime()} directly.
      */
     public static class Builder implements io.helidon.common.Builder<SerializationConfig> {
-        // TODO change default action to FAIL for 3.0.0
         private Action onWrongConfig = configuredAction(PROP_WRONG_CONFIG_ACTION, Action.WARN);
-        // TODO change default action to CONFIGURE for 3.0.0
         private Action onNoConfig = configuredAction(PROP_NO_CONFIG_ACTION, Action.WARN);
         private String filterPattern = System.getProperty(PROP_PATTERN);
         private TraceOption traceSerialization = configuredTrace(TraceOption.NONE);

diff --git a/...f_ok/src/test/java/io/helidon/tests/integration/jep290/checkffok/DeserializationTest.java b/...f_ok/src/test/java/io/helidon/tests/integration/jep290/checkffok/DeserializationTest.java
@@ -37,7 +37,6 @@
 
 class DeserializationTest {
     private static final String TEST_STRING = "Hello_" + new Random().nextInt(10);
-    ;
 
     @BeforeAll
     static void configureDeserialization() {