You are viewing a plain text version of this content. The canonical link for it is here.

Posted to scm@geronimo.apache.org by rm...@apache.org on 2019/09/23 06:56:10 UTC

svn commit: r1867367 - /geronimo/specs/trunk/geronimo-json_1.1_spec/src/main/java/javax/json/EmptyJsonObject.java

Author: rmannibucau
Date: Mon Sep 23 06:56:10 2019
New Revision: 1867367

URL: http://svn.apache.org/viewvc?rev=1867367&view=rev
Log:
Patch from @mkarg on github (https://github.com/apache/geronimo-specs/pull/15/files), fixing EmptyJsonObject implementation

Modified:
    geronimo/specs/trunk/geronimo-json_1.1_spec/src/main/java/javax/json/EmptyJsonObject.java

Modified: geronimo/specs/trunk/geronimo-json_1.1_spec/src/main/java/javax/json/EmptyJsonObject.java
URL: http://svn.apache.org/viewvc/geronimo/specs/trunk/geronimo-json_1.1_spec/src/main/java/javax/json/EmptyJsonObject.java?rev=1867367&r1=1867366&r2=1867367&view=diff
==============================================================================
--- geronimo/specs/trunk/geronimo-json_1.1_spec/src/main/java/javax/json/EmptyJsonObject.java (original)
+++ geronimo/specs/trunk/geronimo-json_1.1_spec/src/main/java/javax/json/EmptyJsonObject.java Mon Sep 23 06:56:10 2019
@@ -49,12 +49,12 @@ class EmptyJsonObject extends AbstractMa
 
     @Override
     public String getString(String name) {
-        return null;
+        throw new NullPointerException("Calling getString on EmptyJsonObject");
     }
 
     @Override
     public String getString(String name, String defaultValue) {
-        return null;
+        return defaultValue;
     }
 
     @Override
@@ -79,7 +79,7 @@ class EmptyJsonObject extends AbstractMa
 
     @Override
     public boolean isNull(String name) {
-        return true;
+        throw new NullPointerException("Calling isNull on EmptyJsonObject");
     }
 
     @Override