* From: "Ingo Molnar" <[email protected]>
>
> * Zhaolei <[email protected]> wrote:
>
>> It it because I use similar format with other TRACE_EVENT
>> definition. Actually, I don't know why we need to add prefix and
>> postfix TABs, and is it necessary to use tab to make alignment for
>> each fields.
>
> The __array() entry is a bit special - i wouldnt worry much about
> it, please ignore the checkpatch warning in this case - or remove
> the prefix tab.
Hello, Ingo
Thanks for your teach.
I think remove the prefix tab only(but remain postfix) make code looks
a bit ugly. I prefer to remain both tabs.
It is to say I will not modify tabs in this patch.
Thanks
Zhaolei
>
> The alignment of the fields are useful visually: all the field names
> are enumerated in a vertical row and are easy to see at a glance.
> The postfix tabs are nice because they move the non-C-syntax closing
> ')' out of the line of sight. The prefix tabs are useful for a
> similar reason.
>
> Ingo
>
>????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay? ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?