Re: Analysis of clang results against the nmap-npingchanges branch for nping directory only.

[David Fifield <david () bamsoftware com>]

On Tue, Jun 19, 2012 at 11:20:25AM -0400, James Rogers wrote:
> Hi!
>
> I added the no return attribute to several function definitions in
> nmap-npingchanges/nping/output.h.  Then ran the clang analyzer against
> nmap-npingchanges and got a report with many less bugs.
>
> This totally eliminated the rest of the clang bug reports for
> nmap-npingchanges/nping/.
>
> I've attached a diff to this email, please review and see if you want
> me to check these changes in.

It looks good to me, except that the indentation is off in one line.

David Fifield
_______________________________________________
Sent through the nmap-dev mailing list
http://cgi.insecure.org/mailman/listinfo/nmap-dev
Archived at http://seclists.org/nmap-dev/