TODO comment breaks ABAP Doc

[ConjuringCoffee]

Hi Jörg-Michael,

I found an example where the inserted TODO comment for unused variables from rule "Delete unused variables" breaks ABAP Doc comments.

Before:

  METHOD example.
    "! Test
    DATA lv_test TYPE i.
  ENDMETHOD.

After:

  METHOD example.
    "! Test
    " TODO: variable is never used (ABAP cleaner)
    DATA lv_test TYPE i.
  ENDMETHOD.

The TODO comment separates the ABAP Doc comment from the variable, causing a syntax warning:

ABAP Doc comment is in the wrong position.

[jmgrassau]

Hi ConjuringCoffee,

ah, good point, that's not yet considered, thanks for reporting this!

Kind regards,
Jörg-Michael

[jmgrassau]

Hi ConjuringCoffee,

thanks again – this bug should be fixed with the next release:

Cleanup of obsolete TODO comments should work even from wrong positions.

Kind regards,
Jörg-Michael

P.S.: Code for further tests:

  METHOD any_method.
    "! Test
    DATA lv_test TYPE i.
  ENDMETHOD.

  METHOD any_method2.
    " comment
    "! Test 1
    "! Test 2
    DATA lv_test TYPE i.
  ENDMETHOD.

  METHOD other_method.
    " TODO: variable is never used (ABAP cleaner)
    "! Test 1
    "! Test 2
    DATA lv_test TYPE i.
    RETURN lv_test.
  ENDMETHOD.

  METHOD other_method2.
    "! Test
    " TODO: variable is never used (ABAP cleaner)
    DATA lv_test TYPE i.
    RETURN lv_test.
  ENDMETHOD.

[ConjuringCoffee]

Great, thank you!