Re: [PATCH 11/14] USB: serial: mos7840: move constants to right

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Mathieu OTHACEHE <m.othacehe@xxxxxxxxx> writes:
> On Sun, Jan 31, 2016 at 10:33:54PM +0100, Bjørn Mork wrote:
>
>> I feel I'm missing something here, so I have to ask the stupid question:
>> What could possibly be the benefit here?  Is it faster? Safer?  Easier
>> to read?
>
> I don't think it will be faster or safer, maybe easier to read. But the point is to remove the
> warnings produced by the file scripts/coccinelle/misc/compare_const_fl.cocci when
> running make coccicheck.

Making a tool happy is never going to be a valid reason for anything.
If it were, then you would not be necessary...

Luckily you still are. Your task is adding some intelligence to the
output of the script.  That means that you should look at the output and
describe how it improves the code in a way that makes me understand it.
And I can be pretty slow :)

If you can't do that, then there is a high probability that the script
didn't improve anything.


Bjørn
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux