| Summary: | Code formatting parameter list alignment with leading commas fails | ||
|---|---|---|---|
| Product: | [Applications] kdevelop | Reporter: | Steven T. Hatton <hattons> |
| Component: | Astyle | Assignee: | kdevelop-bugs-null |
| Status: | RESOLVED NOT A BUG | ||
| Severity: | normal | CC: | hceylan |
| Priority: | NOR | ||
| Version First Reported In: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | unspecified | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
Are you talking about the automatic indentation (katepart) or "Edit -> Reformat Source" (KDevelop)? On Thursday 09 September 2004 15:04, Jens Dagerbo wrote: > ------- You are receiving this mail because: ------- > You reported the bug, or are watching the reporter. > > http://bugs.kde.org/show_bug.cgi?id=89140 > > > > > ------- Additional Comments From jens.dagerbo swipnet se 2004-09-09 21:04 > ------- Are you talking about the automatic indentation (katepart) or "Edit > -> Reformat Source" (KDevelop)? Settings->Configure Shortcut->Reformat Source "Configure shortcut"? I don't get it.. but I assume you mean you are using the KDevelop reformatting tool (which is based on the astyle library, btw) (Btw, please don't quote stuff in bug reports, it bloats the webpage ) I meant that is how I set it up. I didn't know what it actually invokes. I just know I mapped it to Tab and my code gets reformatted when I hit Tab. *** Bug 92342 has been marked as a duplicate of this bug. *** The use of the equals sign in the parameter list is confusing astyle. Yup. Still happening with 3.4-RC2. Closing as this is an obviously astyle problem, please file upstream (astyle.sourceforge.net). |
Version: 3.1.0 (using KDE 3.3.0, SuSE) Compiler: gcc version 3.3.3 (SuSE Linux) OS: Linux (i686) release 2.6.5-7.108-default I have been rediscovering the convenience of using leading commas in parameter lists, and member initialization lists. When the indentation works correctly, I find it much easier to work with. Unfortunately, I've noticed a problem that arrises when default values are assigned to the formal parameters. The indentation aligns at the location of the RHS token, not the previous comma. The snippet included here shows the problem. LinearBoxView(Box_Interface* bif , const size_t& offset_ , LinearView* holder_ , const size_t& block_w_ = 10 , const size_t& block_h_ = 10 , const size_t& edge_w_=1 );