Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Align exceptions of method definition #365

Open
goreraks opened this issue Sep 30, 2024 · 1 comment
Open

Align exceptions of method definition #365

goreraks opened this issue Sep 30, 2024 · 1 comment

Comments

@goreraks
Copy link

Current Behaviour
Here is the sample source code after applying AbapCleaner,
METHODS my_test_method FOR TESTING RAISING cx_exception_1 cx_exception_2 cx_exception_3 cx_exception_4 cx_exception_5.

Expectation
When we use quick assist(Ctrl+1) for "Add raising declarations", the source code is formatted as below. I think the same must be applied instead of keeping them in a line.

    METHODS my_test_method FOR TESTING
      RAISING
        cx_exception_1 
        cx_exception_2 
        cx_exception_3 
        cx_exception_4 
        cx_exception_5.
@jmgrassau
Copy link
Member

Hi Rakshith,

good point, I think we should add options for this, so you could choose

  • whether or not RAISING and/or the exception class names should continue on the same line (similar to the options in cleanup rule "Align METHODS declarations")
  • whether or not the exception classes should be listed below each other (similar to the options in cleanup rule "Align CLEAR:, FREE: and SORT")

Kind regards,
Jörg-Michael

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants