Merge pull request #351 from vert-x3/issues/nan

Adding support for special doubles

Author: pmlopes

src/main/java/io/vertx/redis/client/Response.java

@@ -93,7 +93,17 @@ default Number toNumber() {
   default @Nullable Double toDouble() {
     final String msg = toString();
     if (msg != null) {
-      return Double.parseDouble(msg);
+      switch (msg) {
+        case "-nan":
+        case "nan":
+          return Double.NaN;
+        case "inf":
+          return Double.POSITIVE_INFINITY;
+        case "-inf":
+          return Double.NEGATIVE_INFINITY;
+        default:
+          return Double.parseDouble(msg);
+      }
     }
     return null;
   }

src/main/java/io/vertx/redis/client/impl/ReadableBuffer.java

@@ -113,8 +113,12 @@ Number readNumber(int end, NumericType type) {
         case DECIMAL:
           if (bytes.length == 3 && bytes[0] == 'i' && bytes[1] == 'n' && bytes[2] == 'f') {
             number = Double.POSITIVE_INFINITY;
+          } else if (bytes.length == 3 && bytes[0] == 'n' && bytes[1] == 'a' && bytes[2] == 'n') {
+              number = Double.NaN;
           } else if (bytes.length == 4 && bytes[0] == '-' && bytes[1] == 'i' && bytes[2] == 'n' && bytes[3] == 'f') {
             number = Double.NEGATIVE_INFINITY;
+          } else if (bytes.length == 4 && bytes[0] == '-' && bytes[1] == 'n' && bytes[2] == 'a' && bytes[3] == 'n') {
+            number = Double.NaN;
           } else {
             number = Double.parseDouble(new String(bytes, StandardCharsets.US_ASCII));
           }

src/test/java/io/vertx/test/redis/RedisClient7Test.java

@@ -254,4 +254,17 @@ public void testBooleanVarArgs(TestContext should) {
           });
       });
   }
+
+  @Test
+  public void testNaN(TestContext should) {
+    final Async test = should.async();
+    final String key = makeKey();
+
+    client.send(cmd(EVAL).arg("return tostring(0/0)").arg(0))
+      .onFailure(should::fail)
+      .onSuccess(ok -> {
+        should.assertTrue(Double.isNaN(ok.toDouble()));
+        test.complete();
+      });
+  }
 }