Bug 169415 - wrong indentation level chosen after multi-line-function call
Summary: wrong indentation level chosen after multi-line-function call
Status: RESOLVED FIXED
Alias: None
Product: kate
Classification: Applications
Component: indentation (show other bugs)
Version: 4.0
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: ---
Assignee: KWrite Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-19 00:59 UTC by Andreas Pakulat
Modified: 2010-02-16 13:51 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
file with multi-line-calls (14.31 KB, text/x-c++src)
2008-08-19 00:59 UTC, Andreas Pakulat
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Pakulat 2008-08-19 00:59:21 UTC
I'll add a file where this happens. The file has a few multi-line function calls like this:

KDevelop::VcsJob* job = iface->diff( url, url,
                               KDevelop::VcsRevision::createSpecialRevision( KDevelop::VcsRevision::Working ),
                               KDevelop::VcsRevision::createSpecialRevision( KDevelop::VcsRevision::Head ) );

And when I hit enter after the semicolon kate puts the cursor on the same level as the arguments instead of on the level of the first "KDevelop" in the first line.

As I've chosen to re-indent code pasted via clipboard kate wrongly formats all lines following such a multi-line-call
Comment 1 Andreas Pakulat 2008-08-19 00:59:45 UTC
Created attachment 26927 [details]
file with multi-line-calls
Comment 2 Dominik Haumann 2010-02-15 01:48:39 UTC
SVN commit 1090245 by dhaumann:

fix: wrong indentation level chosen after multi-line-function call
@apaku: can you please test and confirm that it works?
BUG: 169415

 M  +3 -3      cstyle.js  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1090245
Comment 3 Andreas Pakulat 2010-02-15 09:37:39 UTC
Might take a bit, as I don't have a current trunk checkout around and am not sure wether I want trunk or 4.4 on my main dev-machine :)
Comment 4 Andreas Pakulat 2010-02-16 13:51:59 UTC
Seems you got lucky :) Confirmed, works just fine now, thanks a lot.