Hurricup/bugfixes #2892
Hurricup/bugfixes #2892
Conversation
Qodana for JVM2760 new problems were found
💡 Qodana analysis was run in the pull request mode: only the changed files were checked Detected 11 dependenciesThird-party software listThis page lists the third-party software dependencies used in perl5
Contact Qodana teamContact us at [email protected]
|
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| assertInstanceOf(projects.get(0), MojoApp.class); | ||
| LOG.info("Got: " + projects.get(0)); | ||
| assertInstanceOf(projects.getFirst(), MojoApp.class); | ||
| LOG.info("Got: " + projects.getFirst()); |
Check notice
Code scanning / QDJVM
Non-distinguishable logging calls Note test
| assertInstanceOf(projects.get(0), MojoPlugin.class); | ||
| LOG.info("Got: " + projects.get(0)); | ||
| assertInstanceOf(projects.getFirst(), MojoPlugin.class); | ||
| LOG.info("Got: " + projects.getFirst()); |
Check notice
Code scanning / QDJVM
Non-distinguishable logging calls Note test
| else { | ||
| return PerlDocUtil.getPerlFuncDocFromText(element, subName); | ||
| return switch (element) { | ||
| case null -> null; |
Check notice
Code scanning / QDJVM
Duplicate branches in 'switch' Note documentation
| return completionProcessor.result(); | ||
| return switch (element) { | ||
| case PerlImplicitSubDefinition implicitSubDefinition | ||
| when implicitSubDefinition.isAnonymous() -> completionProcessor.result(); |
Check notice
Code scanning / QDJVM
Duplicate branches in 'switch' Note
| @@ -144,7 +144,7 @@ | |||
| } | |||
| else if (LIST_MODIFYING_EXPR.contains(parentExpressionType)) { | |||
| List<PsiElement> children = PerlArrayUtil.collectListElements(expression); | |||
| if (children.size() > 0 && PsiTreeUtil.isAncestor(children.get(0), originalElement, false)) { | |||
| if (children.size() > 0 && PsiTreeUtil.isAncestor(children.getFirst(), originalElement, false)) { | |||
Check warning
Code scanning / QDJVM
'size() == 0' can be replaced with 'isEmpty()' Warning
| @@ -31,7 +31,7 @@ | |||
| */ | |||
| default @Nullable PsiElement getReferenceExpression() { | |||
| List<PsiElement> unpackedChildren = PerlArrayUtil.collectChildrenList(this); | |||
| return unpackedChildren.size() > 0 ? unpackedChildren.get(0) : null; | |||
| return unpackedChildren.size() > 0 ? unpackedChildren.getFirst() : null; | |||
Check warning
Code scanning / QDJVM
'size() == 0' can be replaced with 'isEmpty()' Warning
| } | ||
| return null; | ||
| return switch (element) { | ||
| case null -> null; |
Check notice
Code scanning / QDJVM
Duplicate branches in 'switch' Note documentation
| } | ||
| return null; | ||
| return switch (element) { | ||
| case null -> null; |
Check notice
Code scanning / QDJVM
Duplicate branches in 'switch' Note
| builder.append("\n").append("Its ").append(conditionalInstruction.getResult()). | ||
| append(" branch, condition: ").append(getTextSafe(conditionalInstruction.getCondition())); | ||
| switch (instruction) { | ||
| case PerlIteratorConditionInstruction ignored -> { |
Check notice
Code scanning / QDJVM
Duplicate branches in 'switch' Note test
|
No description provided.