[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 104533] peculiar c-style indentation after indented parameter
From:       <sigra () home ! se>
Date:       2007-08-31 17:18:01
Message-ID: 20070831171801.24828.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=104533         




------- Additional Comments From sigra home se  2007-08-31 19:18 -------
Leo Savernik skrev:
> ------- Additional Comments From l.savernik aon at  2007-08-31 18:32 -------
>   
>> The current behaviour is *not* arbitrary. The program actually tries to
>> do something sensible with the strange stuff it sees. It indents 2
>> spaces more than the preceding line, just as (I assume) it was configured.
>>     
> Distributing long parameter lists over more than line is too common to disregard it as a corner 
> case that demands no action.
>   

Of course it is common to wrap long parameter lists. There is no other
way to make them fit within a certain column limit, which is often a
requirement. But it makes no sense to keep the '{' on the line together
with the last parameter. That makes it a corner case.

The opening '{' has no particular bound to the *last* parameter so there
is no reason to keep those 2 together while splitting the rest. It makes
more sense to keep the parameters together, since they make up a whole
(the parameter list). So the logical point of wrapping is between the
parameter list and the '{'.
[prev in list] [next in list] [prev in thread] [next in thread]