the inevitable C formatting/brace style flamewar

[Jean-Tiare LE BIGOT]

Hi,

Since the flamewar is already going on :p, is it OK not put a space 
after '(' nor
before ')' in test and calls?

I find this more readable whitout the spaces...

Le 2012-08-16 20:12, Bjoern Michaelsen a écrit :
> Hi Michael,
>
> first: Thanks for keeping an old StarDivision tradition alive!
>
> On Thu, Aug 16, 2012 at 07:07:38PM +0200, Michael Stahl wrote:
>>  apparently somebody thought it a great idea to mandate Allman style 
>> for
>
> Someone has an evil grin on his face.
>
>> 1) if the condition is long, then put the opening brace on a new 
>> line.
>>
>> rationale: in this case it's good to visually separate the condition
>> from the compound statement, e.g. this is a PITA to read:
>>
>> >                             if ( !pFly->Lower() || 
>> !pFly->Lower()->IsNoTxtFrm() ||
>> >                                  
>> !((SwNoTxtFrm*)pFly->Lower())->HasAnimation())
>> >                                 pFly->RefreshLaySubsidiary( pPage, 
>> rRect );
>>
>> >         if (bCalledFromShell && !lcl_IsItemSet(*pNewTxtNd, 
>> RES_PARATR_ADJUST) &&
>> >             SFX_ITEM_SET == pAnchorNode->GetSwAttrSet().
>> >             GetItemState(RES_PARATR_ADJUST, sal_True, &pItem))
>> >             static_cast<SwCntntNode 
>> *>(pNewTxtNd)->SetAttr(*pItem);
>
> IMHO, these monster conditions deserve local static inline functions
> with a name
> that says what the heck they are testing for anyway.
>
> Best,
>
> Bjoern
>
> P.S.: Here is a fish for the trollbait: <0())))<
> _______________________________________________
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice

-- 
Jean-Tiare LE BIGOT