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/16 10:11:59 UTC

svn commit: r1339049 - in /avro/trunk: CHANGES.txt lang/c++/impl/avrogencpp.cc

Author: thiru
Date: Wed May 16 08:11:58 2012
New Revision: 1339049

URL: http://svn.apache.org/viewvc?rev=1339049&view=rev
Log:
Missing 'inline' for union set function

Modified:
    avro/trunk/CHANGES.txt
    avro/trunk/lang/c++/impl/avrogencpp.cc

Modified: avro/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/avro/trunk/CHANGES.txt?rev=1339049&r1=1339048&r2=1339049&view=diff
==============================================================================
--- avro/trunk/CHANGES.txt (original)
+++ avro/trunk/CHANGES.txt Wed May 16 08:11:58 2012
@@ -49,6 +49,8 @@ Avro 1.7.0 (unreleased)
     AVRO-1065. NodeRecord::isValid() treats records with no fields as invalid. (thiru)
 Avro 1.6.3 (5 March 2012)
 
+    AVRO-1077. Missing 'inline' for union set function. (thiru)
+
   IMPROVEMENTS
 
     AVRO-1030. Fix a broken link in the documentation.

Modified: avro/trunk/lang/c++/impl/avrogencpp.cc
URL: http://svn.apache.org/viewvc/avro/trunk/lang/c%2B%2B/impl/avrogencpp.cc?rev=1339049&r1=1339048&r2=1339049&view=diff
==============================================================================
--- avro/trunk/lang/c++/impl/avrogencpp.cc (original)
+++ avro/trunk/lang/c++/impl/avrogencpp.cc Wed May 16 08:11:58 2012
@@ -272,7 +272,8 @@ static void generateGetterAndSetter(ostr
         << "    return boost::any_cast<" << type << " >(value_);\n"
         << "}\n\n";
 
-    os << "void" << sn << "set_" << name
+    os << "inline\n"
+        << "void" << sn << "set_" << name
         << "(const " << type << "& v) {\n"
         << "    idx_ = " << idx << ";\n"
         << "    value_ = v;\n"