Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[JENKINS-26156] Calling BodyInvoker.withDisplayName(null) was always …
…a no-op, and will soon be a FindBugs warning.

(cherry picked from commit 7806a76)
  • Loading branch information
jglick committed May 12, 2016
1 parent e3ca5ce commit a14ab59
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 3 deletions.
Expand Up @@ -73,7 +73,6 @@ public static final class Execution extends AbstractStepExecutionImpl {
StepContext context = getContext();
context.newBodyInvoker()
.withCallback(new Callback())
.withDisplayName(null)
.start();
return false;
}
Expand Down
Expand Up @@ -84,7 +84,6 @@ public static class Execution extends AbstractStepExecutionImpl {
listener.getLogger().println("Running in " + dir);
body = getContext().newBodyInvoker()
.withContext(dir)
.withDisplayName(null)
// Could use a dedicated BodyExecutionCallback here if we wished to print a message at the end ("Returning to ${cwd}"):
.withCallback(BodyExecutionCallback.wrap(getContext()))
.start();
Expand Down
Expand Up @@ -25,7 +25,6 @@ public boolean start() throws Exception {
StepContext context = getContext();
body = context.newBodyInvoker()
.withCallback(new Callback())
.withDisplayName(null) // hide the body block
.start();
long now = System.currentTimeMillis();
end = now + step.getUnit().toMillis(step.getTime());
Expand Down

0 comments on commit a14ab59

Please sign in to comment.