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

Apache Commons Collections #21

Open
rdifrango opened this issue Jan 8, 2015 · 5 comments
Open

Apache Commons Collections #21

rdifrango opened this issue Jan 8, 2015 · 5 comments

Comments

@rdifrango
Copy link

Should there be provisions in the configuration to look for usages from Apache Commons Collections, like functions, closures, etc.?

@gaul
Copy link
Owner

gaul commented Jan 8, 2015

@rdifrango Definitely! Could you submit a pull request for more Apache Commons checks? I only have a little familiarity with commons-collections since I mostly use Guava.

@rdifrango
Copy link
Author

@andrewgaul We use it sparingly as we've started to switch over to Guava, so I'll give it a shot.

@gaul
Copy link
Owner

gaul commented Jan 9, 2015

@rdifrango Switching to Guava is another interesting use case -- Modernizer could provide a second optional resource file to suggest Commons replacements. Modernizer only supports a single resource file now but supporting more is easy to add.

@Stephan202
Copy link
Contributor

+1 for the idea of generalizing the plugin to facilitate migrations from old lib X to newer lib Y (where in this case X = Apache Commons and Y is Guava). I guess X and Y could even be different versions of the same library. (Though that's a less useful scenario, provided the library maintainers mark methods with a more suitable replacement @Deprecated.)

@gaul
Copy link
Owner

gaul commented Jun 23, 2015

For example, suggest replacing org.apache.commons.io.input.BoundedInputStream with com.google.common.io.ByteStreams.limit

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

No branches or pull requests

3 participants