[U-Boot] Question about Coding-Style
Chris Moore
moore at free.fr
Mon Feb 10 11:16:38 CET 2014
Hi Albert,
Le 10/02/2014 10:58, Albert ARIBAUD a écrit :
> Hi Tom,
>
> On Tue, 4 Feb 2014 10:07:32 -0500, Tom Rini<trini at ti.com> wrote:
>
>> On Tue, Feb 04, 2014 at 04:02:56PM +0100, Stefano Babic wrote:
>>> Hi Hannes,
>>>
>>> On 04/02/2014 15:50, Hannes Petermaier wrote:
>> [snip]
>>>> Another thing is linewrapping of output strings, to obey to the rules i
>>>> have to format the string as following:
>>>>
>>>> if (i2c_probe(TPS65217_CHIP_PM)) {
>>>> printf("PMIC chip (0x%02x) not present! skipping" \
>>>> "further configuration.\n", TPS65217_CHIP_PM);
>>>> return;
>>>> }
>>>>
>>>> But this makes it impossible to grep the code in case of an error.
>>> You must combine a more complicate grep, maybe with the -A (after
>>> context) option or using a regexp. However, this is not a reason to
>>> break the rule.
>> Strings are the reason to break the rule and we have checkpatch patched
>> (mostly?) to not complain. It's even true in the kernel.
> Hmm... Last time I checked,
>
> "abc"
> "def"
>
> Is a valid C string, and does not require a backslash. Do I miss
> something?
Yes, you are of course correct: the result is the C string "abcdef" and
the backslash in the original is superfluous.
However a grep for "abcdef" in the source code won't match it :(
This is why splitting strings like this is discouraged in the Linux kernel.
Cheers,
Chris
---
Ce courrier électronique ne contient aucun virus ou logiciel malveillant parce que la protection avast! Antivirus est active.
http://www.avast.com
More information about the U-Boot
mailing list