Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-7818

Suspicious condition

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • New

    Description

      Hi
      Please, look this code fragment:

          else if (q1 instanceof AndQueryNode)
            op = ANDOperation.Q1;
          else if (q1 instanceof AndQueryNode)
            op = ANDOperation.Q2;
      

      (q1 instanceof AndQueryNode) is checked twice.

      Probably it should be:

          else if (q1 instanceof AndQueryNode)
            op = ANDOperation.Q1;
          else if (q2 instanceof AndQueryNode)
            op = ANDOperation.Q2;
      

      This possible defect found by AppChecker

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            uschindler Uwe Schindler
            AppChecker AppChecker
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment