[PATCH v2] staging: vt6656: Remove useless function
Dan Carpenter
dan.carpenter at oracle.com
Fri Dec 23 07:45:58 UTC 2011
On Wed, Dec 21, 2011 at 10:12:13AM -0200, Marcos Souza wrote:
> > > /*
> > > static const iw_handler iwctl_handler[] =
> > > {
> > > - (iw_handler) iwctl_commit, // SIOCSIWCOMMIT
> > > (iw_handler) iwctl_giwname, // SIOCGIWNAME
> >
> > This is the same as before. The other comment was just a style
> > issue but this change completely breaks everything. :/ You can't
> > just shift all the array members off by one.
> >
>
> The other removed reference was a commented array. I just removed the
> reference to the function and don't removed the entire commented array for
> don't mix different issues.
>
> If you saw the source file, you'll see this part of the code that this
> array is commented.
>
Ah. I see.
Changes to commented code can't break things, obviously. I'd
normally prefer it if people didn't introduce breakage into
commented code. This particular breakage especially is subtle and
easily avoidable by just putting a NULL as the first element.
You mentioned that it wasn't appropriate to remove the comment in
this patch. You are planning to remove it soon though, yes?
regards,
dan carpenter
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/attachments/20111223/50a31b3a/attachment.asc>
More information about the devel
mailing list