Re: Linux Coding style template

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

 



On Sun, Aug 07, 2016 at 05:53:15PM +0300, Charls D. Chap wrote:
> On Sun, Aug 7, 2016 at 7:39 AM, Greg KH <greg@xxxxxxxxx> wrote:
> > On Sun, Aug 07, 2016 at 12:22:03AM +0300, Charls D. Chap wrote:
> >> I want to transform some C source files. following Linux kernel coding style.
> 
> My questions are more generic, i am interesting in an automatic way for
> identation!

Why?  It's best to do it "by hand" after using a few basic indent rules,
as Lindent provides.

> And also, i am interesting to what kernel developers will
> do in the following cases

Why?

> please take a look to this example
> width is 90, what a kernel developer will do? spliti in two lines, or
> leave it us it is,
> cause i think it's more readable, in this way
> qq->run = mmap(NULL, mmap_size, PROT_READ | PROT_WRITE, MAP_SHARED,
> qq->vcpu, 0);

That doesn't look readable at all, are you sure you ment it to look like
that?

> Also, have you find a way to tell indent not to indent, struct definitions?
> In .h, when you have a huge structure definition, it's more readable,
> to have it's struct friend one below the other, aligned!
> struct kvm_mem_bank {
>           struct list_head           list;
>          u64                              guest_phys_addr;
>          void                              *host_addr;
>         u64                               size;
> };

That's not aligned at all :(

> This also counts for variables initiliazation
> int x = 0
> char  *dog         = animal->dog->name;
> char  *cat          = animal->cat->name;
> char * elephant = animal->elephant->name;
> 
> Any ideas?

Don't do the above, it looks horrible.

> - Does script/Lindent STRICTLY follows all linux kernel coding style rules?

No, it gets you close.

> - What scripts/templates do you use/suggest for elegant code style
> with the fact that
>   we are talking about Linux Kernel Coding?

Use an editor with the proper settings and then fix it up by hand.

Or even better yet, just always write your code using the proper
settings.

> The GNU guys suggest indent with the following parameters:
> -nbad -bap -nbc -bbo -hnl -br -brs -c33 -cd33 -ncdb -ce -ci4
> -cli0 -d0 -di1 -nfc1 -i8 -ip0 -l80 -lp -npcs -nprs -npsl -sai
> -saf -saw -ncs -nsc -sob -nfca -cp33 -ss -ts8 -il1

Where do they say that?

> On the other hand:
> Linux/script/Lindent contains:
> -npro -kr -i8 -ts8 -sob -l80 -ss -ncs -cp1

And the differnces are what?

> Differences so far:
> 1)
> indent with gnu parameters inserts a space on labels eg.
>  err: vs
> err:

Don't put a space.

> 2)
> indent with gnu parameters inserts 26 spaces
> #endif                                                          /*
> something like this                 */

That doesn't make sense, does it?

> So probably gnu site is outdated?

What gnu site?

You do know _why_ we have a coding style, right?  It's not about being
pedantic, it's about your brain.

thanks,

greg k-h

_______________________________________________
Kernelnewbies mailing list
Kernelnewbies@xxxxxxxxxxxxxxxxx
https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies



[Index of Archives]     [Newbies FAQ]     [Linux Kernel Mentors]     [Linux Kernel Development]     [IETF Annouce]     [Git]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux SCSI]     [Linux ACPI]
  Powered by Linux