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

Hurricup/bugfixes 2 #2922

Merged
merged 11 commits into from
Oct 20, 2024
Merged

Hurricup/bugfixes 2 #2922

merged 11 commits into from
Oct 20, 2024

Conversation

hurricup
Copy link
Collaborator

Code style and modernization

@hurricup hurricup enabled auto-merge (rebase) October 20, 2024 08:43
Copy link

Qodana for JVM

3489 new problems were found

Inspection name Severity Problems
Usage of API marked for removal 🔴 Failure 24
Unused declaration 🔶 Warning 749
Missorted modifiers 🔶 Warning 613
Pattern variable can be used 🔶 Warning 407
Deprecated API usage 🔶 Warning 257
Missing '@Override' annotation 🔶 Warning 252
Live templates i18n 🔶 Warning 159
Method parameter always has the same value 🔶 Warning 87
Method can be made 'void' 🔶 Warning 79
Hardcoded strings 🔶 Warning 48
Method always returns the same value 🔶 Warning 45
@NotNull/@Nullable problems 🔶 Warning 34
Can use bounded wildcard 🔶 Warning 26
Class is exposed outside of its visibility scope 🔶 Warning 25
Static initialization in extension point implementations 🔶 Warning 25
Unnecessary semicolon 🔶 Warning 21
Declaration can have 'final' modifier 🔶 Warning 16
Non-platform TokenSet declared in ParserDefinition 🔶 Warning 16
'@RunWith' annotation already exists in a parent class 🔶 Warning 15
Redundant 'throws' clause 🔶 Warning 13
Field may be 'final' 🔶 Warning 9
C-style array declaration 🔶 Warning 7
'final' method in 'final' class 🔶 Warning 3
Missing required 'lang' attribute 🔶 Warning 3
Raw use of parameterized class 🔶 Warning 3
Incorrect string capitalization 🔶 Warning 2
A service can be converted to a light one 🔶 Warning 2
Explicit type can be replaced with '<>' 🔶 Warning 1
Lambda can be replaced with method reference 🔶 Warning 1
Copy of existing static method body 🔶 Warning 1
Plugin.xml dynamic plugin verification 🔶 Warning 1
'protected' member in 'final' class 🔶 Warning 1
Unnecessary conversion to 'String' 🔶 Warning 1
Unnecessary modifier 🔶 Warning 1
Duplicate branches in 'switch' ◽️ Notice 510
Link with unencrypted protocol ◽️ Notice 14
Non-distinguishable logging calls ◽️ Notice 8
Method can be extracted ◽️ Notice 5
Class can be record class ◽️ Notice 2
Immutable collection creation can be replaced with collection factory call ◽️ Notice 2
'equals()' expression replaceable by 'Objects.equals()' expression ◽️ Notice 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Detected 11 dependencies

Third-party software list

This page lists the third-party software dependencies used in perl5

Dependency Version Licenses
ant-launcher 1.10.13 Apache-2.0
W3C-19980720
ant 1.10.13 Apache-2.0
W3C-19980720
com.jetbrains.performanceplugin 242.20224.300+62 Apache-2.0
com.jetbrains.performanceplugin 242.20224.300+630 Apache-2.0
docker 2024.2+808 Apache-2.0
jflex 1.9.2 BSD-3-Clause
jsr305 3.0.2 Apache-2.0
tanvd.grazi 242.20224.300+518 Apache-2.0
LGPL-2.0-or-later
LGPL-2.1-only
MIT
tanvd.grazi 242.20224.300+822 Apache-2.0
LGPL-2.0-or-later
LGPL-2.1-only
MIT
testng-j 242.20224.300+3 MIT
testng-j 242.20224.300+337 MIT
Contact Qodana team

Contact us at [email protected]

@@ -111,7 +112,7 @@
}

public static void reindexProjectFile(Project project, VirtualFile virtualFile) {
if (VfsUtil.isAncestor(project.getBaseDir(), virtualFile, false)) {
if (VfsUtilCore.isAncestor(project.getBaseDir(), virtualFile, false)) {

Check warning

Code scanning / QDJVM

Deprecated API usage Warning

'getBaseDir()' is deprecated
PerlCallObjectValue value = (PerlCallObjectValue)o;

return mySuperContext != null ? mySuperContext.equals(value.mySuperContext) : value.mySuperContext == null;
return mySuperContext != null ? mySuperContext.equals(callObjectValue.mySuperContext) : callObjectValue.mySuperContext == null;

Check notice

Code scanning / QDJVM

'equals()' expression replaceable by 'Objects.equals()' expression Note

mySuperContext != null ? mySuperContext.equals(callObjectValue.mySuperContext) : callObjectValue.myS... replaceable by 'Objects.equals()' expression
PerlValueResolver resolver = (PerlValueResolver)o;

if (!myResolveScope.equals(resolver.myResolveScope)) {
if (!myResolveScope.equals(((PerlValueResolver)o).myResolveScope)) {

Check warning

Code scanning / QDJVM

Pattern variable can be used Warning

Cast expression can be replaced with pattern variable
return false;
}
return Objects.equals(myContextFile, resolver.myContextFile);
return Objects.equals(myContextFile, ((PerlValueResolver)o).myContextFile);

Check warning

Code scanning / QDJVM

Pattern variable can be used Warning

Cast expression can be replaced with pattern variable
@hurricup hurricup merged commit 3d9cce7 into master Oct 20, 2024
38 checks passed
@hurricup hurricup deleted the hurricup/bugfixes-2 branch October 20, 2024 10:22
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 70%)

See analysis details on SonarCloud

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

Successfully merging this pull request may close these issues.

1 participant