Skip to content

Commit

Permalink
[JENKINS-26156] Calling BodyInvoker.withDisplayName(null) was always …
Browse files Browse the repository at this point in the history
…a no-op, and will soon be a FindBugs warning.
  • Loading branch information
jglick committed Apr 8, 2016
1 parent 5c291a9 commit 0841b3e
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -466,7 +466,6 @@ private final class PlaceholderExecutable implements ContinuableExecutable {
listener.getLogger().println("Running on " + computer.getDisplayName() + " in " + workspace); // TODO hyperlink
context.newBodyInvoker()
.withContexts(exec, computer, env, workspace)
.withDisplayName(null)
.withCallback(new Callback(cookie, lease))
.start();
LOGGER.log(FINE, "started {0}", cookie);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,6 @@ public boolean start() throws Exception {
body = getContext().newBodyInvoker()
.withContext(workspace)
.withCallback(new Callback(lease))
.withDisplayName(null)
.start();
return false;
}
Expand Down

0 comments on commit 0841b3e

Please sign in to comment.