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.

Sign up for GitHub

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

Jump to bottom

Nonnegative.Checker class should be static #15

Open

GoogleCodeExporter opened this issue Apr 22, 2015 · 1 comment

Labels

auto-migrated Priority-Medium Type-Defect

GoogleCodeExporter commented Apr 22, 2015

What steps will reproduce the problem?

1. Open Nonnegative.java
2. Look at inner class Checker

What is the expected output? What do you see instead?

Class Checker should be static, just like all other checkers

What version of the product are you using? On what operating system?

r43

Original issue reported on code.google.com by [email protected] on 30 Jun 2009 at 12:50

The text was updated successfully, but these errors were encountered:

Author

GoogleCodeExporter commented Apr 22, 2015

Checker classes are static and public by default (because they're inner types 
of an
interface) but IMHO the source code should be consistent: use static or drop it
everywhere.

Original comment by [email protected] on 18 Jul 2009 at 2:39

GoogleCodeExporter added Priority-Medium Type-Defect auto-migrated labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment