"Warning: comparison between signed and unsigned integer expressions [-Wsign-compare]"
Changed 'unsigned int' into 'Eigen::Index' type for several loop indexes...
"Warning: comparison between signed and unsigned integer expressions [-Wsign-compare]"
Changed 'unsigned int' into 'Eigen::Index' type for several loop indexes...
"Warning: Comparison between signed and unsigned integer expressions [-Wsign-compare]"
Fixed by declaring loop index as library-provided type 'Eigen::Index' instead of raw 'unsigned int'.
"Warning: comparison between signed and unsigned integer expressions [-Wsign-compare]"
Changed 'unsigned int' into 'Eigen::Index' type for several loop indexes...
I'm still finding typos in the source code. I propose that you keep this PR open prior ro relaese and then merge so that if I find other typos in the meantime they will be part this release.
+ some are doxy others are just comment code.
+ some minor whitespace and grammatical tweaks.
+ app/SCL/* typos have also been submitted upstream (https://github.com/stepcode/stepcode/pull/366). So it's ok to merge them in to master.
Since getTypeName() returns None for None and default both must be handled the same way in other switches.
After applying following bug will be fixed:
http://www.freecadweb.org/tracker/view.php?id=2212
Warnings wsa:
src/Mod/Robot/App/kdl_cp/chaindynparam.cpp|105 col 5| warning: control may reach end of non-void function [-Wreturn-type]
src/Mod/Robot/App/kdl_cp/chainidsolver_recursive_newton_euler.cpp|82 col 5| warning: control may reach end of non-void function [-Wreturn-type]
- (error) va_list '<var>' was opened but not closed by va_end()
- (style) Same expression on both sides of '||'
- (style) Same expression on both sides of '!='
