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.

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

[EC75] Useless rule - deprecation #246

Closed
csauldubois opened this issue Nov 23, 2023 · 1 comment · Fixed by #259
Closed

[EC75] Useless rule - deprecation #246

csauldubois opened this issue Nov 23, 2023 · 1 comment · Fixed by #259
Assignees
@dedece35
Labels
🗃️ rule rule improvment or rule development or bug java

Comments

@csauldubois
Copy link

Concatenated strings are automatically optimized by the compiler, in case of a single string or a string appended in a loop. This behaviour is described in the following page :

https://medium.com/javarevisited/java-compiler-optimization-for-string-concatenation-7f5237e5e6ed

Saying that, I think this rule is useless.
@dedece35 dedece35 self-assigned this Dec 3, 2023
@dedece35
Copy link
Member

dedece35 commented Dec 8, 2023 •
edited
Loading

As confirmed with core-team, this rule will be deprecated soon because not applicable for Java since jdk8.
check status of PR #259

@dedece35 dedece35 linked a pull request Dec 10, 2023 that will close this issue
[EC75] Deprecation of rule EC75 for java #259
Merged
This was referenced Dec 10, 2023
Closed
Merged
@dedece35 dedece35 changed the title [EC75] Useless rule [EC75] Useless rule - deprecation Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dedece35 dedece35

Labels
🗃️ rule rule improvment or rule development or bug java
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[EC75] Deprecation of rule EC75 for java green-code-initiative/creedengo-rules-specifications
2 participants
@dedece35 @csauldubois