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

Fake pull-request to showcase the checks #174

Closed
wants to merge 7 commits into from

test PR workflow for spotbugs (hopefully for the last time2)

fbe7989
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Closed

Fake pull-request to showcase the checks #174

test PR workflow for spotbugs (hopefully for the last time2)
fbe7989
Select commit
Loading
Failed to load commit list.
GitHub Actions / SpotBugs failed May 17, 2024 in 0s

SpotBugs Source Code Analyzer report

1 violation(s) found

Annotations

Check warning on line 120 in gama.core/src/gama/gaml/multi_criteria/MulticriteriaAnalyzeOperator.java

See this annotation in the file changed.

@github-actions github-actions / SpotBugs

RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT

Return value of java.util.List.size() ignored, but method has no side effect
Raw output
This code calls a method and ignores the return value. However, our analysis shows that the method (including its implementations in subclasses if any) does not produce any effect other than return value. Thus, this call can be removed.

We are trying to reduce the false positives as much as possible, but in some cases this warning might be wrong. Common false-positive cases include:

- The method is designed to be overridden and produce a side effect in other projects which are out of the scope of the analysis.

- The method is called to trigger the class loading which may have a side effect.

- The method is called just to get some exception.

If you feel that our assumption is incorrect, you can use a @CheckReturnValue annotation to instruct SpotBugs that ignoring the return value of this method is acceptable.