DPDK patches and discussions
 help / color / mirror / Atom feed
* Re: [dpdk-dev] [PKTGEN] OK to reindent the pktgen (mix of tabs and spaces, etc.)?
@ 2016-01-03 17:09 Wiles, Keith
  2016-01-03 22:07 ` Matthew Hall
  2016-01-03 23:35 ` Ferruh Yigit
  0 siblings, 2 replies; 6+ messages in thread
From: Wiles, Keith @ 2016-01-03 17:09 UTC (permalink / raw)
  To: Matthew Hall, dev


On 1/3/16, 2:06 AM, "Matthew Hall" <mhall@mhcomputing.net> wrote:

>Hello,
>
>In over a month I didn't get a reply if it would be OK to clean up the 
>inconsistent tabs and spaces indents in the pktgen, to make it easier 
>for the community to read the code and help maintain it and add bug fixes.
>
>It would be very helpful if I could get a response and a plan for this 
>if my proposal is not acceptable, or it's not going to be very pleasant 
>to patch any bugs I find when testing it, as some of the stuff I'd want 
>to work on in the pktgen comes from files with weird indentation that is 
>not consistent and thus painful to send patches.

Sorry, Matthew, You sent your email around Thanksgiving/release time and I missed it. If you have questions or comments directly for Pktgen please put [PKTGEN] in the subject line I have a filter for that one.
>
>Sincerely,
>Matthew.
>
>On 11/22/15 11:10 PM, Matthew Hall wrote:
>> I would like to reindent it using the following astyle command, with a few
>> small hand edits past that level, to get it closer to most other DPDK code as
>> the inconsistent mix of tabs, spaces, etc. makes it difficult to read and
>> debug when it has issues.
>>
>> Obviously the upstream Lua and common/wr_* code would not be included as they
>> seem to be copied from elsewhere w/ different upstream standards.
>>
>> If I were to make the big diff needed for this, would this item be acceptable
>> upstream?
>>
>> Otherwise it is going to be hard to get more people working on the code
>> reliably as it will be hard for others to follow besides the original authors.
>>
>> astyle \
>> --max-code-length=132 \
>> --style=attach \
>> --break-closing-brackets \
>> --add-brackets \
>> --indent=force-tab=8 \

Pktgen is setup for tabs for 4 (with replace tabs with spaces), using tab stop of 8 is just wrong IMO :-)
Just started using kdevelop instead of eclipse, so I may have corrupted the style some :-(

At least it is suppose to be done that way. I will reformat the code (with tabs=4) and have a look at the output.
The rules from other places like to replace tabs with spaces, which is OK I guess.

I can run the astyle on the code and look at the output, if it looks OK I will submit it to the repo, then we just need to understand your hand edits.



------------------------
We need to discuss the DPDK coding style not in this email, but as a standalone email on the dev list. But here are my thoughts for now.

A bigger question is what is the coding style of DPDK and where is it documented? I looked in the docs/web site and did not find any coding style suggestions. Maybe I missed it someplace.

Also using something like astyle or uncrustify tool would be a good thing to be done before releases or a patch is submitted.

I know checkpatch likes to check for some items like extra white space characters at end of lines. I like tools that do just one job like astyle or uncrustify or lint tools. IMO checkpatch should not be doing style checks per say, but a couple of minor checks is fine. If we want the code to run through a style checker then lets start using one.


++Keith
>> --indent-switches \
>> --indent-labels \
>> --indent-col1-comments \
>> --pad-oper \
>> --pad-header \
>> --unpad-paren \
>> --align-pointer=type \
>> --align-reference=type \
>> --suffix=none \
>> --lineend=linux \
>> ./app/**/*.{c,h}
>>
>> Sincerely,
>> Matthew.
>>
>


Regards,
Keith





^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-01-04  2:35 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-03 17:09 [dpdk-dev] [PKTGEN] OK to reindent the pktgen (mix of tabs and spaces, etc.)? Wiles, Keith
2016-01-03 22:07 ` Matthew Hall
2016-01-03 23:06   ` Stephen Hemminger
2016-01-04  2:35   ` Wiles, Keith
2016-01-03 23:35 ` Ferruh Yigit
2016-01-04  0:15   ` Wiles, Keith

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).