Messages in this thread Patch in this message | | | Date | Fri, 6 Jun 2008 16:30:37 -0700 | From | Greg KH <> | Subject | Re: update checkpatch.pl to version 0.19 |
| |
On Fri, Jun 06, 2008 at 07:22:25PM +0000, Linux Kernel Mailing List wrote: > @@ -1920,23 +2004,16 @@ sub process { > WARN("kfree(NULL) is safe this check is probabally not required\n" . $hereprev); > } > } > -# check for needless usb_free_urb() checks > - if ($prevline =~ /\bif\s*\(([^\)]*)\)/) { > - my $expr = $1; > - if ($line =~ /\busb_free_urb\(\Q$expr\E\);/) { > - WARN("usb_free_urb(NULL) is safe this check is probabally not required\n" . $hereprev); > - } > - } >
I'm curious as to why this check was removed. Any specific reason? It was valid from what I can tell.
thanks,
greg k-h
| |