Details

    • Number of attachments :
      0

      Issue Links

        Activity

        Hide
        Fabrice Bellingard added a comment -

        Works well, but I've noticed a false-positive that could be avoided IMO: the following line is detected as commented-out code

        // --------------------------------------------------------------------
        

        For example, on "org/as3commons/lang/ClassUtils.as" of "AS3Commons Lang" (lines 353 and 357).

        There's also the false-positive due to the last semi-colon of the ASL v2 header, but I don't know if something can be done on this case for now ( ?).

        Show
        Fabrice Bellingard added a comment - Works well, but I've noticed a false-positive that could be avoided IMO: the following line is detected as commented-out code // -------------------------------------------------------------------- For example, on "org/as3commons/lang/ClassUtils.as" of "AS3Commons Lang" (lines 353 and 357). There's also the false-positive due to the last semi-colon of the ASL v2 header, but I don't know if something can be done on this case for now ( ?).
        Hide
        Evgeny Mandrikov added a comment -

        Fabrice, we have exactly the same limitations in plugins for other languages, so in fact I would prefer to postpone this for now and sort out later for all plugins. What is your feeling?

        Show
        Evgeny Mandrikov added a comment - Fabrice, we have exactly the same limitations in plugins for other languages, so in fact I would prefer to postpone this for now and sort out later for all plugins. What is your feeling?
        Hide
        Fabrice Bellingard added a comment -

        OK, I'm fine with this, as long as we create a ticket for the Flex plugin so that users can see that it's a known limitation.

        Show
        Fabrice Bellingard added a comment - OK, I'm fine with this, as long as we create a ticket for the Flex plugin so that users can see that it's a known limitation.
        Hide
        Fabrice Bellingard added a comment -

        Manually tested

        Show
        Fabrice Bellingard added a comment - Manually tested
        Hide
        Evgeny Mandrikov added a comment -

        Ticket created - SONARPLUGINS-1956

        Show
        Evgeny Mandrikov added a comment - Ticket created - SONARPLUGINS-1956
        Hide
        Evgeny Mandrikov added a comment -

        Closed without integration test

        Show
        Evgeny Mandrikov added a comment - Closed without integration test

          People

          • Assignee:
            Evgeny Mandrikov
            Reporter:
            Evgeny Mandrikov
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: