Uploaded image for project: 'logback'
  1. logback
  2. LOGBACK-789

Static code analysis reveals not fully covered enum in ch.qos.logback.core.subst.Tokenizer

    XMLWordPrintable

Details

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Minor Minor
    • None
    • None
    • None
    • None

    Description

      Static code analysis reveals not fully covered enum in ch.qos.logback.core.subst.Tokenizer

      The DEFAULT_VAL_STATE is not being covered in

      switch (state)

      { case LITERAL_STATE: addLiteralToken(tokenList, buf); break; case START_STATE: throw new ScanException("Unexpected end of pattern string"); }

      return tokenList;
      }

      Probably not a problem, but I suggest adding a

      case DEFAULT_VAL_STATE:
      throw new Error("Cannot happen");
      break;

      Attachments

        Activity

          People

            logback-dev Logback dev list
            dtonhofer David Tonhofer
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated: