[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Drivemap module
From: |
Marco Gerards |
Subject: |
Re: [PATCH] Drivemap module |
Date: |
Tue, 05 Aug 2008 13:31:28 +0200 |
User-agent: |
Gnus/5.110006 (No Gnus v0.6) Emacs/21.4 (gnu/linux) |
Hi,
Javier Martín <address@hidden> writes:
>> Anyway, since "they" are more likely to maintain the code in
>> the long run than you, in general, the question is whether
>> the code is more likely to become buggy by their hacking on
>> it, if it follows project coding style or someone else's
>> (your) "safer" coding style. Likely it's safer if using a
>> consistent programming style. Although I personally don't
>> see that it's very helpful to have a style that makes things
>> down to the order of "==" arguments be consistent within the
>> project; haphazard only slows reading the tiniest bit, and I
>> don't think it makes a different what order the arguments are...
>
> Hmm... I was partially expecting a flamefest to start. Pity ^^
> Well, let's spill a little napalm: the GNU style bracing is extremely
> silly!! Why the hell are the "if" and "else" blocks indented differently
> in this example?
> if (condition)
> return 0;
> else
> {
> return -1;
> }
> Nah, I'm not really bringing that issue, I was just joking, and in fact
> I'm reconsidering my objections to the operator== arguments order rule,
> even though I still consider my style safer and more sensible. If
> someone else wants to express their opinion on that issue, do it fast
> before I completely submit to Master Marco's will :D
Please don't be sarcastic, start flame wars or call names. It will not
help anyone.
--
Marco
- Re: [PATCH] Drivemap module, Javier Martín, 2008/08/03
- Re: [PATCH] Drivemap module, Marco Gerards, 2008/08/05
- Re: [PATCH] Drivemap module, Javier Martín, 2008/08/05
- Re: [PATCH] Drivemap module, Javier Martín, 2008/08/09
- Re: [PATCH] Drivemap module, Marco Gerards, 2008/08/13
- Re: [PATCH] Drivemap module, Javier Martín, 2008/08/13
- Re: [PATCH] Drivemap module, Robert Millan, 2008/08/13
- Re: [PATCH] Drivemap module, Javier Martín, 2008/08/13
- Re: [PATCH] Drivemap module, Marco Gerards, 2008/08/13