diff --git a/Source/Modules/csharp.cxx b/Source/Modules/csharp.cxx index d715aadb2..619fb7ebd 100644 --- a/Source/Modules/csharp.cxx +++ b/Source/Modules/csharp.cxx @@ -925,7 +925,7 @@ public: if ((throw_parm_list = Getattr(n, "catchlist"))) { Swig_typemap_attach_parms("throws", throw_parm_list, f); for (p = throw_parm_list; p; p = nextSibling(p)) { - if ((tm = Getattr(p, "tmap:throws"))) { + if (Getattr(p, "tmap:throws")) { canThrow(n, "throws", p); } } @@ -3747,7 +3747,7 @@ public: if (throw_parm_list) Swig_typemap_attach_parms("throws", throw_parm_list, 0); for (p = throw_parm_list; p; p = nextSibling(p)) { - if ((tm = Getattr(p, "tmap:throws"))) { + if (Getattr(p, "tmap:throws")) { if (gencomma++) { Append(w->def, ", "); Append(declaration, ", "); diff --git a/Source/Modules/d.cxx b/Source/Modules/d.cxx index 29f96e548..a1a454bd0 100644 --- a/Source/Modules/d.cxx +++ b/Source/Modules/d.cxx @@ -1695,7 +1695,7 @@ public: if ((throw_parm_list = Getattr(n, "catchlist"))) { Swig_typemap_attach_parms("throws", throw_parm_list, f); for (p = throw_parm_list; p; p = nextSibling(p)) { - if ((tm = Getattr(p, "tmap:throws"))) { + if (Getattr(p, "tmap:throws")) { canThrow(n, "throws", p); } } @@ -2224,7 +2224,7 @@ public: if (throw_parm_list) Swig_typemap_attach_parms("throws", throw_parm_list, 0); for (p = throw_parm_list; p; p = nextSibling(p)) { - if ((tm = Getattr(p, "tmap:throws"))) { + if (Getattr(p, "tmap:throws")) { if (gencomma++) { Append(w->def, ", "); Append(declaration, ", "); diff --git a/Source/Modules/java.cxx b/Source/Modules/java.cxx index bfa02d7e2..d59ca8bd6 100644 --- a/Source/Modules/java.cxx +++ b/Source/Modules/java.cxx @@ -1016,7 +1016,7 @@ public: if ((throw_parm_list = Getattr(n, "catchlist"))) { Swig_typemap_attach_parms("throws", throw_parm_list, f); for (p = throw_parm_list; p; p = nextSibling(p)) { - if ((tm = Getattr(p, "tmap:throws"))) { + if (Getattr(p, "tmap:throws")) { addThrows(n, "tmap:throws", p); } } @@ -3631,7 +3631,7 @@ public: SwigType *adjustedreturntype = covariant ? covariant : returntype; Parm *adjustedreturntypeparm = NewParmNode(adjustedreturntype, n); - if ((tm = Swig_typemap_lookup("directorin", adjustedreturntypeparm, "", 0)) + if (Swig_typemap_lookup("directorin", adjustedreturntypeparm, "", 0) && (cdesc = Getattr(adjustedreturntypeparm, "tmap:directorin:descriptor"))) { // Note that in the case of polymorphic (covariant) return types, the @@ -3658,7 +3658,7 @@ public: } String *jdesc = NULL; - if ((tm = Swig_typemap_lookup("directorin", tp, "", 0)) + if (Swig_typemap_lookup("directorin", tp, "", 0) && (jdesc = Getattr(tp, "tmap:directorin:descriptor"))) { // Objects marshalled passing a Java class across JNI boundary use jobject - the nouse flag indicates this @@ -3911,7 +3911,7 @@ public: if (throw_parm_list) Swig_typemap_attach_parms("throws", throw_parm_list, 0); for (p = throw_parm_list; p; p = nextSibling(p)) { - if ((tm = Getattr(p, "tmap:throws"))) { + if (Getattr(p, "tmap:throws")) { addThrows(n, "tmap:throws", p); if (gencomma++) { diff --git a/Source/Modules/lang.cxx b/Source/Modules/lang.cxx index 144638366..002a1fdd7 100644 --- a/Source/Modules/lang.cxx +++ b/Source/Modules/lang.cxx @@ -1134,7 +1134,6 @@ int Language::globalfunctionHandler(Node *n) { int Language::callbackfunctionHandler(Node *n) { Swig_require("callbackfunctionHandler", n, "name", "*sym:name", "*type", "?value", NIL); - String *symname = Getattr(n, "sym:name"); String *type = Getattr(n, "type"); String *name = Getattr(n, "name"); String *parms = Getattr(n, "parms"); diff --git a/Source/Modules/octave.cxx b/Source/Modules/octave.cxx index f78c71fd3..7d27c1095 100644 --- a/Source/Modules/octave.cxx +++ b/Source/Modules/octave.cxx @@ -1319,7 +1319,7 @@ public: if (throw_parm_list) Swig_typemap_attach_parms("throws", throw_parm_list, 0); for (p = throw_parm_list; p; p = nextSibling(p)) { - if ((tm = Getattr(p, "tmap:throws"))) { + if (Getattr(p, "tmap:throws")) { if (gencomma++) { Append(w->def, ", "); Append(declaration, ", "); diff --git a/Source/Modules/php.cxx b/Source/Modules/php.cxx index 1f2078a1e..986911a12 100644 --- a/Source/Modules/php.cxx +++ b/Source/Modules/php.cxx @@ -1177,11 +1177,6 @@ public: if (!o) { // This "overloaded method" is really just one with default args. really_overloaded = false; - if (l != full_parmlist) { - l = full_parmlist; - if (wrapperType == memberfn) - l = nextSibling(l); - } } } @@ -2472,7 +2467,7 @@ done: if (throw_parm_list) Swig_typemap_attach_parms("throws", throw_parm_list, 0); for (p = throw_parm_list; p; p = nextSibling(p)) { - if ((tm = Getattr(p, "tmap:throws"))) { + if (Getattr(p, "tmap:throws")) { if (gencomma++) { Append(w->def, ", "); Append(declaration, ", "); diff --git a/Source/Modules/python.cxx b/Source/Modules/python.cxx index 56107e0c0..5dc2cb8be 100644 --- a/Source/Modules/python.cxx +++ b/Source/Modules/python.cxx @@ -4583,7 +4583,7 @@ int PYTHON::classDirectorMethod(Node *n, Node *parent, String *super) { if (throw_parm_list) Swig_typemap_attach_parms("throws", throw_parm_list, 0); for (p = throw_parm_list; p; p = nextSibling(p)) { - if ((tm = Getattr(p, "tmap:throws"))) { + if (Getattr(p, "tmap:throws")) { if (gencomma++) { Append(w->def, ", "); Append(declaration, ", "); diff --git a/Source/Modules/r.cxx b/Source/Modules/r.cxx index ab13cb7fc..078b2576c 100644 --- a/Source/Modules/r.cxx +++ b/Source/Modules/r.cxx @@ -572,7 +572,7 @@ String * R::createFunctionPointerHandler(SwigType *t, Node *n, int *numArgs) { Parm *p = parms; for (i = 0; p; p = nextSibling(p), ++i) { String *arg = Getattr(p, "name"); - String *lname = NewString(""); + String *lname; if (!arg && Cmp(Getattr(p, "type"), "void")) { lname = NewStringf("s_arg%d", i+1); diff --git a/Source/Modules/ruby.cxx b/Source/Modules/ruby.cxx index e16bbc9ba..ff8d70d46 100644 --- a/Source/Modules/ruby.cxx +++ b/Source/Modules/ruby.cxx @@ -737,10 +737,6 @@ private: String *autodoc = Getattr(n, "feature:autodoc"); autodoc_l dlevel = autodoc_level(autodoc); - symname = Getattr(n, "sym:name"); - if ( Getattr( special_methods, symname ) ) - symname = Getattr( special_methods, symname ); - switch (dlevel) { case NO_AUTODOC: case NAMES_AUTODOC: @@ -3093,7 +3089,7 @@ public: if (throw_parm_list) Swig_typemap_attach_parms("throws", throw_parm_list, 0); for (p = throw_parm_list; p; p = nextSibling(p)) { - if ((tm = Getattr(p, "tmap:throws"))) { + if (Getattr(p, "tmap:throws")) { if (gencomma++) { Append(w->def, ", "); Append(declaration, ", ");