[Swig-devel] [ swig-Patches-3284326 ] Fix for Tcl wrapper compilation warning

[SourceForge.net <no...@so...>]

Patches item #3284326, was opened at 2011-04-11 05:28
Message generated for change (Comment added) made by delbaere
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=301645&aid=3284326&group_id=1645

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Colin McDonald (cjmcdonald)
Assigned to: Nobody/Anonymous (nobody)
Summary: Fix for Tcl wrapper compilation warning

Initial Comment:
Compiling swig 2.0.3 Tcl wrappers with Visual C++ on Windows
now gives the following warning:

warning C4018: '<' : signed/unsigned mismatch

This patch fixes the problem:


--- Lib/tcl/tclinit.swg.rev12563        Tue Mar 29 19:57:42 2011
+++ Lib/tcl/tclinit.swg Mon Apr 11 10:14:26 2011
@@ -71,7 +71,7 @@

 SWIGINTERN void
 SWIG_Tcl_InstallMethodLookupTables(Tcl_Interp *interp) {
-  int i;
+  size_t i;

   for (i = 0; i < swig_module.size; ++i) {
     swig_type_info *type = swig_module.type_initial[i];
{


----------------------------------------------------------------------

Comment By: Christian Delbaere (delbaere)
Date: 2011-04-11 13:23

Message:
Looks like the "interp" parameter could also be removed from
SWIG_Tcl_InstallMethodLookupTables() to prevent an unused variable warning
with some compiler settings.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=301645&aid=3284326&group_id=1645