Details

    • Type: New Feature New Feature
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.2
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Description

      • Title
        Avoid empty block
      • Description
        In most cases, en empty block represents a missing implementation.

      examples :

      if(something){   /*Non-Compliant*/
         //What ?
      }
      
      for (var i=0;i<cars.length;i++) { /*Non-Compliant*/
         //Empty on purpose or missing piece of code ?
      }
      
      • Violation message
        Provide the missing piece of code
      • Default severity
        Major

      Part of Sonar way quality profile

        Issue Links

          Activity

          Hide
          Fabrice Bellingard added a comment -

          Empty function blocks should be considered as well.

          Show
          Fabrice Bellingard added a comment - Empty function blocks should be considered as well.
          Hide
          Evgeny Mandrikov added a comment -

          Fixed.

          Show
          Evgeny Mandrikov added a comment - Fixed.
          Hide
          Fabrice Bellingard added a comment -

          Manually tested.

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

          Integration tests were updated.

          Show
          Evgeny Mandrikov added a comment - Integration tests were updated.
          Hide
          Evgeny Mandrikov added a comment -

          In fact we decided to revert change on functions as it generates false-positives.

          Show
          Evgeny Mandrikov added a comment - In fact we decided to revert change on functions as it generates false-positives.

            People

            • Assignee:
              Evgeny Mandrikov
              Reporter:
              Freddy Mallet
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: