Uploaded image for project: 'WildFly Elytron'
  1. WildFly Elytron
  2. ELY-737

Coverity static analysis: Operands don't affect result in Elytron

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • Minor
    • 1.1.0.Beta15
    • None
    • None

    Description

      Coverity static-analysis scan found unnecessary code in the DigestQuote.quoteNeeded() method on line 43 (currently).
      One part of the condition is ch >= 0, which is always true, because the ch parameter has char type.

      https://scan7.coverity.com/reports.htm#v16159/p11778/fileInstanceId=5760146&defectInstanceId=1541357&mergedDefectId=1375729

      Suggested improvement
      Remove the useless ch >= 0 from the expression.

      Attachments

        Issue Links

          clones

          Bug - A problem that impairs or prevents the functions of the product. JBEAP-7062 Coverity static analysis: Operands don't affect result in Elytron

          • Minor - To be worked after urgent, high, and medium priorities are resolved. This term is chosen when the severity of the issue is low, or the complexity or effort to correct it may be higher, relatively speaking. For low priority issues, known workarounds exist or are not needed due to the trivial effort needed to address the issue.
          • Verified

          Activity

            People

              rhn-support-ivassile Ilia Vassilev
              josef.cacek@gmail.com Josef Cacek (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: