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

Posted to commits@avro.apache.org by th...@apache.org on 2012/05/13 19:28:19 UTC

svn commit: r1337946 - in /avro/trunk: CHANGES.txt lang/c++/api/NodeImpl.hh

Author: thiru
Date: Sun May 13 17:28:18 2012
New Revision: 1337946

URL: http://svn.apache.org/viewvc?rev=1337946&view=rev
Log:
AVRO-1065. NodeRecord::isValid() treats records with no fields as invalid

Modified:
    avro/trunk/CHANGES.txt
    avro/trunk/lang/c++/api/NodeImpl.hh

Modified: avro/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/avro/trunk/CHANGES.txt?rev=1337946&r1=1337945&r2=1337946&view=diff
==============================================================================
--- avro/trunk/CHANGES.txt (original)
+++ avro/trunk/CHANGES.txt Sun May 13 17:28:18 2012
@@ -44,6 +44,7 @@ Avro 1.7.0 (unreleased)
 
     AVRO-1066. ArrayIndexOutOfBoundsException in ParsingEncoder when trying to use a json encoder to serialize a deep object graph. (thiru)
 
+    AVRO-1065. NodeRecord::isValid() treats records with no fields as invalid. (thiru)
 Avro 1.6.3 (5 March 2012)
 
   IMPROVEMENTS

Modified: avro/trunk/lang/c++/api/NodeImpl.hh
URL: http://svn.apache.org/viewvc/avro/trunk/lang/c%2B%2B/api/NodeImpl.hh?rev=1337946&r1=1337945&r2=1337946&view=diff
==============================================================================
--- avro/trunk/lang/c++/api/NodeImpl.hh (original)
+++ avro/trunk/lang/c++/api/NodeImpl.hh Sun May 13 17:28:18 2012
@@ -253,7 +253,6 @@ class AVRO_DECL NodeRecord : public Node
     bool isValid() const {
         return (
                 (nameAttribute_.size() == 1) && 
-                (leafAttributes_.size() > 0) &&
                 (leafAttributes_.size() == leafNameAttributes_.size())
                );
     }