#100 Space after comma

closed-fixed
3 days ago
2011-07-08
Michael S.
No

I can't see that space after comma is working in version 2.02.

Accordingly to previous reported bugs (2047652 and 1539953) it should now be possible to format with space after comma, e.g. this

class Foo
{
public:
void foo(int param1,int param2);
void foo2(int param1,int param2);
}

void Foo::foo(int param1,int param2)
{
foo2(param1,param2);
}

...becomes...

class Foo
{
public:
void foo(int param1, int param2);
void foo2(int param1, int param2);
}

Foo::foo(int param1, int param2)
{
foo2(param1, param2);
}

/Michael

Identical issues (these are having status Deleted):
-------------------------------------------------------------------------
* Insert space after "," - ID: 2047652 (Reported 2008-08- 12)
https://sourceforge.net/tracker/index.php?func=detail&aid=2047652&group_id=2319&atid=102319

* Place space after komma\'s between parameters - ID: 1539953 (Reported 2006-08-14)
https://sourceforge.net/tracker/index.php?func=detail&aid=1539953&group_id=2319&atid=102319

Discussion

  • Jim Pattee

    Jim Pattee - 2012-02-07
    • labels: 640327 --> Malformatting
     
  • David Faure

    David Faure - 2013-12-21

    Well, it works, it inserted a space after the comma.

    What's missing, is to remove spaces before commas.

    I have just implemented this: http://www.davidfaure.fr/kde/astyle_comma.diff

    Please integrate this patch into the next version of astyle.

     
  • David Faure

    David Faure - 2014-12-31

    Ping? Anything I can do to help getting this patch into astyle?

     
  • Jim Pattee

    Jim Pattee - 2015-03-22

    Changes in SVN.
    Unpad before commas.
    New option to pad after commas.

     
  • Jim Pattee

    Jim Pattee - 2015-03-22
    • status: open --> open-fixed
    • assigned_to: Jim Pattee
    • Priority: 5 --> 7
     
  • Jim Pattee

    Jim Pattee - 3 days ago
    • status: open-fixed --> closed-fixed
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks