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

new ListOf<>(array) violates PMD ArrayIsStoredDirectly #1053

Open
dgroup opened this issue Jun 1, 2019 · 3 comments
Open

new ListOf<>(array) violates PMD ArrayIsStoredDirectly #1053

dgroup opened this issue Jun 1, 2019 · 3 comments

Comments

@dgroup
Copy link
Contributor

dgroup commented Jun 1, 2019

On qulice v0.18.19

public App(final String... args) {
  this.args = new ListOf<>(args);
}

gives PMD The user-supplied array 'args' is stored directly. (ArrayIsStoredDirectly).
What's wrong with it?

@0crat
Copy link
Collaborator

0crat commented Jun 1, 2019

@krzyk/z please, pay attention to this issue

@0crat
Copy link
Collaborator

0crat commented Jun 1, 2019

@dgroup/z this project will fix the problem faster if you donate a few dollars to it; just click here and pay via Stripe, it's very fast, convenient and appreciated; thanks a lot!

@pnatashap
Copy link
Contributor

@yegor256 may be really it is better to disable this rule? https://pmd.github.io/pmd/pmd_rules_java_bestpractices.html#arrayisstoreddirectly

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

3 participants