Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

False positive/noise :unused-locals warnings found in cases/testcases/testtest.clj #247

Open
jafingerhut opened this issue Jan 10, 2018 · 0 comments

Comments

@jafingerhut
Copy link
Collaborator

jafingerhut commented Jan 10, 2018

False positive/noise :unused-locals warnings found in cases/testcases/testtest.clj
also some similar ones in cases/testcases/suspicious.clj

Found by enabling :unused-locals linter in linters_test.clj for namespace testcases.testtest.

The last 2 are good warnings. For the first 3, it would be nicer if Eastwood did not report them. Not sure how difficult detecting those cases are.

{{:linter :unused-locals,
   :msg "let bound symbol 'G__<num>' never used",
   :file "testcases/testtest.clj",
   :line 124,
   :column 16}
  1,
  {:linter :unused-locals,
   :msg "let bound symbol 'expr__<num>' never used",
   :file "testcases/testtest.clj",
   :line 125,
   :column 16}
  1,
  {:linter :unused-locals,
   :msg "let bound symbol 'pred__<num>' never used",
   :file "testcases/testtest.clj",
   :line 125,
   :column 16}
  1,
  {:linter :unused-locals,
   :msg "let bound symbol 'x' never used",
   :file "testcases/testtest.clj",
   :line 128,
   :column 27}
  1,
  {:linter :unused-locals,
   :msg "let bound symbol 'x' never used",
   :file "testcases/testtest.clj",
   :line 129,
   :column 24}
  2}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant