Uploaded image for project: 'Plugins'
  1. Plugins
  2. CONTRIB-5891

Detect and error about C-style comments

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Deferred
    • Icon: Minor Minor
    • None
    • 2.7.9, 2.8.7, 2.9.1
    • Local: Code checker
    • None
    • MOODLE_27_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE

      Right now we are not looking for C-style (/* .... */, single or multiline) in the checker. More yet, we are, incorrectly, suggesting them!

      Coding style clearly shows them as incorrect, bad.

      And we have a good number of them:

      $ ag '\/\*$' | wc -l
      3506
      

      Although it's clear they are minority:

      $ ag '\/\**$' | wc -l
      220880
      

      So, this is about to make the checker to detect such incorrect C-style inline comments and report them as errors.

      Ciao

            moodle.com Moodle HQ
            stronk7 Eloy Lafuente (stronk7)
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.