THRIFT-1200 js: JS compiler generates code that clobbers existing namespaces

fix broken build(jslint) introduced with original patch

git-svn-id: https://svn.apache.org/repos/asf/thrift/trunk@1134093 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/compiler/cpp/src/generate/t_js_generator.cc b/compiler/cpp/src/generate/t_js_generator.cc
index f925a4d..8daaedf 100644 (file)
--- a/compiler/cpp/src/generate/t_js_generator.cc
+++ b/compiler/cpp/src/generate/t_js_generator.cc
@@ -277,7 +277,9 @@ void t_js_generator::init_generator() {
   if( ns_pieces.size() > 0){
     for(size_t i = 0; i < ns_pieces.size(); ++i) {
       pns += ((i == 0) ? "" : ".") + ns_pieces[i];
-      f_types_ << "if (typeof " << pns << " === 'undefined') " << pns << " = {};" << endl;
+      f_types_ << "if (typeof " << pns << " === 'undefined') {" << endl;
+        f_types_ << "  " << pns << " = {};" << endl;
+        f_types_ << "}" << endl;
     }
   }