Skip to content

test1.java critical #1

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

test1.java critical #1

wants to merge 5 commits into from

Conversation

deepcodeg
Copy link
Owner

No description provided.

@deepcodeappdev
Copy link

deepcodeappdev bot commented May 6, 2020

DeepCode's analysis on #0ed24b found:

  • 2 critical issues. ⚠️ 2 warnings 2 minor issues. ✔️ 0 issues were fixed.
  • We show critical issues and security vulnerabilities below 👇.

List of critical issues

Description of the issue ❌ Example fixes
compare does not return -1, but needs to. The implementer must ensure sgn(x.compare(y)) == -sgn(y.compare(x)) for all x and y. Occurrences: Example fixes 🔧
The usage of regex in replace will change only the first match. Consider using a regular expression with the global replacement flag (/g). Occurrences: Example fixes 🔧


👉 View analysis in DeepCode’s Dashboard

☺️ If you want to provide feedback on our bot, here is how to contact us.

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

Successfully merging this pull request may close these issues.

1 participant