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
Type inference for generic instance creation #2100
Conversation
* Replacing type parameters with <>
b9241d2
to
1e398c5
Compare
The Java Language Specification and convention states that static ought to be listed before the final modifier (https://docs.oracle.com/javase/specs/jls/se11/html/jls-8.html#jls-8.1.1)
1e398c5
to
5f01d7b
Compare
@ullenius Are these all? Did you look for more codes similar to the changes you have made? |
@sazzad16 I added 3 more commits fixing similar stuff. There are 2 more occurrences in ClusterScriptingCommandsTest that I did not change. And one in JedisTest. I'm somewhat reluctant to modify the tests given that they fail to pass as it is. |
@ullenius Let us know when you're done changing. |
Type inference for generic instance creation.
<T>
with<>
for inferred typeThis feature was added in Java 7 back in 2011 (!). The
pom.xml
explicitly specifies Java 7 so this does not break anything.Minor refactoring:
Removing unused method parameter for private helper method
GenericObjectPoolConfig poolConfig
is passed to the method but never usedDeclaring modifiers in recommended order (
static final)
6fada2eReferences: Oracle docs