Menu ▾ ▴

#408 Raise or remove the limit for --max-instatement-indent parameter

closed-invalid
Jim Pattee
None
2016-11-19
2016-09-26
Eric Chamberland
No

Hi,

I would like to set a higher limit than 79 to --max-instatement-indent parameter.

I would like either "unlimited" or a value of 180.

This will greatly help into introducing this tool into our development with a minimum of code modification (actually, it unalign a lot of our code... :/ )

Thanks,

Eric

Discussion

  • Jim Pattee

    Jim Pattee - 2016-09-29

    The current maximum is 120. It can be changed in astyle_main.cpp

     

  • Eric Chamberland

    Eric Chamberland - 2016-09-29

    Understood.

    To be clear: Do you ask me to change it and ask for a pull request or you are telling me that it will not be changed in the official release?

    Thanks,
    Eric

     

  • Jim Pattee

    Jim Pattee - 2016-09-30

    I haven’t decided if it should be changed. 180 would be off the screen for most displays. But it doesn’t take much to change it.

    It would help if you could send some code so I could see what is happening.

     

  • Eric Chamberland

    Eric Chamberland - 2016-09-30

    Ok, the max of 120 if fine. I was misleaded by the limite of 79 by UniversalIndentGUI which limit is set to 79... I will report this to them!
    Thanks for the information!
    Eric

     

  • Jim Pattee

    Jim Pattee - 2016-11-19
    • status: open --> closed-invalid
    • assigned_to: Jim Pattee
    • Priority: 5 --> 7
     

Log in to post a comment.