Skip to content

Commit

Permalink
Fix a style nit.
Browse files Browse the repository at this point in the history
Summary: Checking for empty should use `isEmpty()` instead of doing math.

Test Plan: Set breakpoints and make sure this condition still gets hit.
  • Loading branch information
oconnor663 committed Jun 11, 2014
1 parent b5e11a6 commit b88b3af
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/com/facebook/buck/cli/MissingSymbolsHandler.java
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ public void onMissingSymbol(MissingSymbolEvent event) {
@Subscribe
public void onBuildFinished(BuildEvent.Finished event) {
// Shortcircuit if there aren't any failures.
if (missingSymbolEvents.get(event.getBuildId()).size() == 0) {
if (missingSymbolEvents.get(event.getBuildId()).isEmpty()) {
return;
}
missingSymbolsHandler.printNeededDependencies(missingSymbolEvents.get(event.getBuildId()));
Expand Down

0 comments on commit b88b3af

Please sign in to comment.