[PATCH] Remove superfluous if(p!=NULL) checks around free(p); p=NULL;

Cyril Brulebois kibi at debian.org
Wed Nov 10 14:39:44 PST 2010


Matt Turner <mattst88 at gmail.com> (10/11/2010):
> Oh, don't apologize. I was merely curious. Maybe someone would
> already know of a semantic patch that's floated around LKML to do
> this. For every patch like this I see, I become less and less
> confident that we've found them all.

I've factorized my semantic patches. The first one (say: "A") should
be able to handle all trivial cases I found up to now. The other one
(say: "B") should be able to find candidates.

Running them against all the input and video drivers maintained in
Debian by debian-x@, it resulted in 15 "A" patches and 4 "B" patches,
following the distribution below:

xf86-input-aiptek
  A
xf86-input-elographics
  A
xf86-input-joystick
  A
xf86-input-mouse
  A
xf86-input-vmmouse
  A
xf86-video-ati
  A
xf86-video-fbdev
  B
xf86-video-intel
  A
xf86-video-nouveau
  A
xf86-video-nv
  A
xf86-video-radeonhd
  A
xf86-video-tga
  A
xf86-video-trident
  A
xf86-video-tseng
  A
xf86-video-vesa
  B
xf86-video-vmware
  A
  B
xf86-video-voodoo
  A
  B

Shall I post a patch for all those, using --in-reply-to to parent them
to a single mail? If so, maybe that one?

Mraw,
KiBi.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg-devel/attachments/20101110/c6b3fb40/attachment.pgp>


More information about the xorg-devel mailing list