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

Posted to commits@commons.apache.org by oh...@apache.org on 2015/04/18 21:06:51 UTC

svn commit: r1674558 - /commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration2/plist/XMLPropertyListConfiguration.java

Author: oheger
Date: Sat Apr 18 19:06:50 2015
New Revision: 1674558

URL: http://svn.apache.org/r1674558
Log:
Fixed another IntelliJ warning related to a potential NPE.

Modified:
    commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration2/plist/XMLPropertyListConfiguration.java

Modified: commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration2/plist/XMLPropertyListConfiguration.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration2/plist/XMLPropertyListConfiguration.java?rev=1674558&r1=1674557&r2=1674558&view=diff
==============================================================================
--- commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration2/plist/XMLPropertyListConfiguration.java (original)
+++ commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration2/plist/XMLPropertyListConfiguration.java Sat Apr 18 19:06:50 2015
@@ -574,6 +574,7 @@ public class XMLPropertyListConfiguratio
             {
                 // remove the root of the XMLPropertyListConfiguration previously pushed on the stack
                 PListNodeBuilder builder = pop();
+                assert builder != null : "Stack was empty!";
                 if (peek() instanceof ArrayNodeBuilder)
                 {
                     // create the configuration