From 32bf6d9937b07c2c523e8895d4f2dffdade42a14 Mon Sep 17 00:00:00 2001
From: Jack Nagel <jacknagel@gmail.com>
Date: Tue, 23 Dec 2014 01:04:44 -0500
Subject: [PATCH] Avoid check-then-act pattern in clear_cache

---
 Library/Homebrew/download_strategy.rb | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/Library/Homebrew/download_strategy.rb b/Library/Homebrew/download_strategy.rb
index 1ca262e787..793a59da65 100644
--- a/Library/Homebrew/download_strategy.rb
+++ b/Library/Homebrew/download_strategy.rb
@@ -28,6 +28,7 @@ class AbstractDownloadStrategy
   # Remove {#cached_location} and any other files associated with the resource
   # from the cache.
   def clear_cache
+    rm_rf(cached_location)
   end
 
   def expand_safe_system_args args
@@ -127,10 +128,6 @@ class VCSDownloadStrategy < AbstractDownloadStrategy
     @clone
   end
 
-  def clear_cache
-    cached_location.rmtree if cached_location.exist?
-  end
-
   def head?
     version.head?
   end
@@ -240,7 +237,8 @@ class CurlDownloadStrategy < AbstractDownloadStrategy
   end
 
   def clear_cache
-    [cached_location, temporary_path].each { |f| f.unlink if f.exist? }
+    super
+    rm_rf(temporary_path)
   end
 
   private