Hi All, Here are some comments about the warnings report:
- The report sent by Sur has 4000 lines. I just spent a long time taking a good look again at it. My opinion does not change. - As pointed the warnings in third party libraries will be ignored. I think Sur could exclude that from the report. Would be easier for me to read. Especially Scintilla and MathGL that are in C++ generates lots of warnings. - Some warnings are well known. Most of them are reported only on GCC, some are reported on Visual C++ too. Thanks for the warnings but in those cases I know what I'm doing, they will be simply ignored. Maybe using a macro could solve those typecasts errors, but I couldn't build one that satisfy GCC and Visual C++. This can be a good contribution. - As pointed most of the other warnings are ignored. Because most of them are cosmetic, so we are helping the tool to report less warnings but we are not actually helping IUP to be a better tool. I simply don't have time to fix this kind of warning. On the other hand, some warnings reported by Coverity Scan last week were actually very useful. I think some memory leaks and even potential memory errors were fixed. *To distinguish those warnings from the ones we can ignore is crucial*. Memory check tools are very time consuming, but sometimes they got very interesting results. We run them sporadically. In Visual C++ I use "Visual Leak Detector". In Linux we use Valgrind. But there are professional tools out there. Does anyone has access to one of these? That would be valuable contribution too. Today I found some useful warnings too at the report, but I had to slowly page down until I found them. Some functions that were defined but not used. They can be errors because we did a lot of stuff and some functions could be simply trash to be removed or could be forgotten to be used. So I will check and commit to the SVN very soon. Thanks for the support folks, but got to focus or we will not be able to fulfill the tasks we have and improve old code. Best, Scuri
_______________________________________________ Iup-users mailing list Iup-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/iup-users
