From b88b3af184703bac09e753faa71e6bf0c2af1b32 Mon Sep 17 00:00:00 2001 From: Jack O'Connor Date: Wed, 11 Jun 2014 14:48:55 -0700 Subject: [PATCH] Fix a style nit. Summary: Checking for empty should use `isEmpty()` instead of doing math. Test Plan: Set breakpoints and make sure this condition still gets hit. --- src/com/facebook/buck/cli/MissingSymbolsHandler.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/com/facebook/buck/cli/MissingSymbolsHandler.java b/src/com/facebook/buck/cli/MissingSymbolsHandler.java index aa3b06273e3..8b113f9b872 100644 --- a/src/com/facebook/buck/cli/MissingSymbolsHandler.java +++ b/src/com/facebook/buck/cli/MissingSymbolsHandler.java @@ -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()));