From 38eea034c330b8a929455a1f49a91c56480771fa Mon Sep 17 00:00:00 2001 From: Jens Geyer Date: Wed, 26 Feb 2014 21:00:05 +0100 Subject: [PATCH] THRIFT-2380 unreachable code (CID 1174546, CID 1174679) Patch: Jens Geyer --- compiler/cpp/src/generate/t_delphi_generator.cc | 10 ---------- compiler/cpp/src/generate/t_go_generator.cc | 7 ------- 2 files changed, 17 deletions(-) diff --git a/compiler/cpp/src/generate/t_delphi_generator.cc b/compiler/cpp/src/generate/t_delphi_generator.cc index 62924a8e..1a51e83e 100644 --- a/compiler/cpp/src/generate/t_delphi_generator.cc +++ b/compiler/cpp/src/generate/t_delphi_generator.cc @@ -3147,12 +3147,7 @@ void t_delphi_generator::generate_delphi_struct_result_writer_impl(ostream& out, if (fields.size() > 0) { indent_impl(code_block) << "field_ := TFieldImpl.Create;" << endl; - bool first = true; for (f_iter = fields.begin(); f_iter != fields.end(); ++f_iter) { - if (! first) { - indent_impl(code_block) << "end else" << endl; - } - indent_impl(code_block) << "if (__isset_" << prop_name(*f_iter,is_exception) << ") then" << endl; indent_impl(code_block) << "begin" << endl; indent_up_impl(); @@ -3168,11 +3163,6 @@ void t_delphi_generator::generate_delphi_struct_result_writer_impl(ostream& out, indent_impl(code_block) << "oprot.WriteFieldEnd();" << endl; indent_down_impl(); } - - if (! first) { - indent_impl(code_block) << "end;" << endl; - } - } diff --git a/compiler/cpp/src/generate/t_go_generator.cc b/compiler/cpp/src/generate/t_go_generator.cc index 6b90cabf..af7b3798 100644 --- a/compiler/cpp/src/generate/t_go_generator.cc +++ b/compiler/cpp/src/generate/t_go_generator.cc @@ -2092,13 +2092,6 @@ void t_go_generator::generate_service_remote(t_service* tservice) indent() << "argvalue" << i << " := containerStruct" << i << "." << argName << endl; } else { throw ("Invalid argument type in generate_service_remote"); - string err1(tmp("err")); - f_remote << - indent() << "argvalue" << i << ", " << err1 << " := eval(flag.Arg(" << flagArg << "))" << endl << - indent() << "if " << err1 << " != nil {" << endl << - indent() << " Usage()" << endl << - indent() << " return" << endl << - indent() << "}" << endl; } if (the_type->is_typedef()) { -- 2.17.1