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

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

XMLWordPrintable

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

      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.

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

              Created:
              Updated:
              Resolved: