In the isclosebrace() function, the 'c' variable is compared to the '}' character twice. PVS-Studio's diagnostic rule: V501 There are identical sub-expressions to the left and to the right of the '||' operator: c = L'}' || c = L'}' Merge ccrystaleditview.cpp 1556 UINT cchText = cbData / sizeof(TCHAR) - 1 UINT cbData = (UINT) ::GlobalSize (hData) We just wanted to show that the PVS-Studio analyzer is quickly developing and learning to detect more new types of bugs. Note that many of the errors described here refer not to the WinMerge project itself but the libraries it uses. Let's get back to the defects found in the code. The cheapest way to detect many of them is to use static code analyzers. New errors appear in the program while writing new code.Every new analyzer's version usually contains new diagnostic rules, which means that you can detect more errors.There is no sense in checking a project with a static code analysis tool only once and get satisfied with it. But what is the most important, there is the following conclusion to draw from it: Below are the results of this second check. Ī year has passed since then, and we have decided to test the new version of WinMerge with the new version of PVS-Studio. There were few errors whose description can be found in the article "Comparing the general static analysis in Visual Studio 2010 and PVS-Studio by examples of errors detected in five open source projects". We checked the WinMerge with it some time ago. The PVS-Studio analyzer allows you to detect errors in C/C++ applications. The article continues the idea that static code analyzers are tools to be used regularly, not once.
0 Comments
Leave a Reply. |
AuthorWrite something about yourself. No need to be fancy, just an overview. ArchivesCategories |