Remove unused pushedManifest local variable in CopyUtils

[sonarqube-agent]

This PR was created because a team member assigned these issues to the Remediation Agent.

Removes an unused local variable declaration in CopyUtils.java that was capturing the return value from pushManifest() without ever being referenced. The method call is still executed for its side effects, eliminating the code smell while maintaining the same functionality.

View Project in SonarCloud

---

Fixed Issues

java:S1481 - Remove this unused "pushedManifest" local variable. • MINOR • View issue

Location: src/main/java/land/oras/CopyUtils.java:244

Why is this an issue?

An unused local variable is a variable that has been declared but is not used anywhere in the block of code where it is defined. It is dead code, contributing to unnecessary complexity and leading to confusion when reading the code. Therefore, it should be removed from your code to maintain clarity and efficiency.

What changed

This hunk removes the unused local variable 'pushedManifest' by changing the assignment statement 'Manifest pushedManifest = target.pushManifest(' to just 'target.pushManifest('. The method call is still executed for its side effects, but the return value is no longer captured in a variable that was never used, eliminating the unused local variable code smell.

--- a/src/main/java/land/oras/CopyUtils.java
+++ b/src/main/java/land/oras/CopyUtils.java
@@ -244,1 +244,1 @@ public final class CopyUtils {
-                    Manifest pushedManifest = target.pushManifest(
+                    target.pushManifest(

Have a suggestion or found an issue? Share your feedback here.

---

SonarQube Remediation Agent uses AI. Check for mistakes.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud