jyapayne/swig
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix some MSVC compiler warnings in the test suite

Browse source 
nested_in_template_wrap.cxx(247): warning C4244: 'initializing': conversion from 'double' to 'int', possible loss of data
python_pybuffer_wrap.cxx(2788): warning C4267: 'return': conversion from 'size_t' to 'int', possible loss of data
Modules\python.cxx(2227) : warning C4800: 'int' : forcing value to bool 'true' or 'false' (performance warning)
This commit is contained in:
Zackery Spytz 2019-03-07 09:44:01 -07:00
commit a0b84f5180
3 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Examples/test-suite/nested_in_template.i
View file

@ -19,7 +19,7 @@ struct OuterTemplate<1>
struct ConcreteDerived : AbstractBase struct ConcreteDerived : AbstractBase
{ {
ConcreteDerived() : m_value(0.) {} ConcreteDerived() : m_value(0) {}
explicit ConcreteDerived(int value) : m_value(value) {} explicit ConcreteDerived(int value) : m_value(value) {}
virtual bool IsSameAs(const AbstractBase& other) const { virtual bool IsSameAs(const AbstractBase& other) const {

2
Examples/test-suite/python_pybuffer.i
View file

@ -28,7 +28,7 @@
++count; ++count;
return count; return count;
} }
int func4(const char *buf4) size_t func4(const char *buf4)
{ {
return strlen(buf4); return strlen(buf4);
} }

2
Source/Modules/python.cxx
View file

@ -2224,7 +2224,7 @@ public:
n = nn; n = nn;
Parm *parms = Getattr(n, "parms"); Parm *parms = Getattr(n, "parms");
bool varargs = parms ? emit_isvarargs(parms) : 0; int varargs = parms ? emit_isvarargs(parms) : 0;
/* We prefer to explicitly list all parameters of the C function in the /* We prefer to explicitly list all parameters of the C function in the
generated Python code as this makes the function more convenient to use, generated Python code as this makes the function more convenient to use,