Uploaded image for project: 'Commons BCEL'
  1. Commons BCEL
  2. BCEL-273

Regressions running FindBugs on BCEL6

VotersStop watchingWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • 6.0
    • 6.0
    • Main
    • None

    Description

      PREFACE

      I'm trying to port FindBugs to the latest greatest BCEL 6 state from trunk, see https://github.com/findbugsproject/findbugs/issues/106.

      In short, FindBugs was using some manually created BCEL 5.2 fork which allowed us somehow run BCEL on Java 8 and even Java 9.

      Unfortunately I have no idea how to rebuild that fork, because no one in the project left any documentation.

      So anyway, my goal was to use unmodified BCEL6, therefore I fixed all compile issues caused by BCEL-222 () and was able to run FB with BCEL6 head.

      Unfortunately I found some regressions.

      Environment

      I'm using my own git clone from BCEL mirror https://github.com/iloveeclipse/commons-bcel/commits/trunk

      The clone only adds few Eclipse files and fixes some obvious errors, nothing worth to mention here. The reason why I'm not using svn because it is too slow and I won't waste my time.

      To see commits I needed to port FB to BCEL6 port, go to https://github.com/findbugsproject/findbugs/tree/java9_bcel6

      How to reproduce
      git clone https://github.com/findbugsproject/findbugs.git
      cd findbugs
      git checkout java9_bcel6
      cd findbugs
      ant
      cd ../findbugsTestCases
      ant
      

      You should run Java 8.
      After running tests, you should see many various errors, like: https://github.com/findbugsproject/findbugs/files/300401/bcel6_test_result.txt

      None of those error appear with the old private patched BCEL5.2 snapshot we've used before.

      It would be nice to fix the errors before the BCEL release.

      Attachments

        1. bcel6_test_result.txt
          13.01 MB
          Andrey Loskutov
        2. BetterCFGBuilder2.java
          50 kB
          Andrey Loskutov
        3. invokedynamic.txt
          2 kB
          David Brosius

        Activity

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

          People

            Unassigned Unassigned
            iloveeclipse Andrey Loskutov
            Votes:
            1 Vote for this issue
            Watchers:
            4 Stop watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment