Skip to contentSkip to conversation

OK to reindent the pktgen (mix of tabs and spaces, etc.)?

DISCUSSION Matthew Hall 1 participant 2 messages Nov 23, 2015 – Jan 3, 2016

lore
Share & tools
lore
#0 lore raw

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 \
--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.

#1 lore raw

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.

Sincerely,
Matthew.

Quoted 35 lines · Matthew Hall

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 \
--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.