search

RIOT-OS / RIOT

RIOT - The friendly OS for IoT
https://riot-os.org
GNU Lesser General Public License v2.1
4.93k stars 1.99k forks source link

astyle: Update docs #3469

Closed jnohlgard closed 8 years ago

jnohlgard commented 9 years ago

According to @authmillenon it should no longer be necessary to run keyword_space.sed since #3384

miri64 commented 9 years ago

I did not test that, just conjectured.

A-Paul commented 9 years ago

Have been curious about that.

1  s/\bif(/if (/
2  s/\bswitch(/switch (/
3  s/\bcase(/case (/
4  s/\bwhile(/while (/
5  s/\bfor(/for (/
6  s/\bdo(/do (/
7  s/! /!/

Lines 1,2,3,4,5 are covered by used astyle parameter --pad-header. Line 6 isn't, as you wouldn't have that do followed by left parenthesis in C likes.
There seems to be nothing in astyle which could perform the substitution in Line 7.

jnohlgard commented 9 years ago

@A-Paul I don't know what you mean with 6, do is a keyword in C.

A-Paul commented 9 years ago

@gebart, yes it is. But followed by left curly bracket and not by left parenthesis. I should have been more specific.

jnohlgard commented 9 years ago

Ah, that's a typo. Good job spotting it!

OlegHahm commented 8 years ago

obsolete