[PATCH 0/8] Improve style of OM files

Nelson Castillo nelsoneci at gmail.com
Thu Oct 16 22:45:12 CEST 2008


On Thu, Oct 16, 2008 at 12:54 PM, Jonas Bonn <jonas.bonn at gmail.com> wrote:
>>
>> The idea is to:
>>
>> - Remove unwanted spaces
>> - Apply coding style fixes
>> - Modify strings for consistency, for instance replace "FIC Neo1073
>> GTA01" with "Openmoko GTA01".
>> - Fix one typo
>>
>>
>
> I don't necessarily agree with all these trivial whitespace changes.
> Two things bug me:
>
> i)  Much of it seems pretty arbitrary (and even inconsistent)...
> linebreaks and braces on if-statements, in particular.

Oh, I forgot to use the if {} else {} rule :-/ Sorry. Some old habits
are hard to forget.
For linebreaks I used the 80-chars rule.
I always make spaces visible. Misplaced spaces/tabs turn red in my
editor :-) I just hate them :-P

> PS:  No offense, Nelson.  The time you are investing in this is
> definitely appeciated... let's just get your time focussed on
> something more meaningful -- it's the best way to learn, really.  Hope
> the above list can be of some inspiration to you...

It's OK. My initial goal was to be comfortable with Stgit and I like it now.
As I've said before I'm getting ready for next month. Now I think I'd better
wait until then.

N.-



More information about the openmoko-kernel mailing list