commit 5df24f7afb05fe25352f1fe5fff0206eb0103a3d
author Roger Meier <roger@apache.org> Wed Nov 13 22:08:06 2013 +0100
committer Roger Meier <roger@apache.org> Wed Nov 13 22:08:06 2013 +0100
tree f31b3b0e18481cf97547b9a40f10effa9475b453
parent b54fbd32cf79bffa0e96409af55783f17abf5aa8

THRIFT-2172 Java compiler allocates optionals array for every struct with an optional field
Patch: Andrew Gaul

compiler/cpp/src/generate/t_java_generator.cc

diff --git a/compiler/cpp/src/generate/t_java_generator.cc b/compiler/cpp/src/generate/t_java_generator.cc
index b8cd930..e978af8 100644
--- a/compiler/cpp/src/generate/t_java_generator.cc
+++ b/compiler/cpp/src/generate/t_java_generator.cc
@@ -1371,7 +1371,7 @@
     }
 
     if (optionals > 0) {
-      std::string output_string = "private _Fields optionals[] = {";
+      std::string output_string = "private static final _Fields optionals[] = {";
       for (m_iter = members.begin(); m_iter != members.end(); ++m_iter) {
         if ((*m_iter)->get_req() == t_field::T_OPTIONAL) {
           output_string = output_string + "_Fields." + constant_name((*m_iter)->get_name()) + ",";