Michael Stefaniuc wrote:
On Mon, Mar 03, 2003 at 07:01:35PM +0100, Eric Pouech wrote:

-    while (*lpszStr);
+    while (*lpszStr)

could we catch this type of error with smatch ?

It's pretty easy to check for "(while|for)(....);", but the interesting part is to keep the false positive minimal. I'll see what i can do.
look for:
if (...);
(while|for)(...);{
};while

should limit false positives

A+
--
Eric Pouech




Reply via email to