no need to assign if we are gonna return

rails · Aug 11, 2010 · bfd7281 · bfd7281
1 parent ff760dd
commit bfd7281
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/actionpack/lib/action_dispatch/http/cache.rb b/actionpack/lib/action_dispatch/http/cache.rb
@@ -113,10 +113,10 @@ def string_body?
         DEFAULT_CACHE_CONTROL = "max-age=0, private, must-revalidate"
 
         def set_conditional_cache_control!
-          control = @cache_control
-
           return if self["Cache-Control"].present?
 
+          control = @cache_control
+
           if control.empty?
             headers["Cache-Control"] = DEFAULT_CACHE_CONTROL
           elsif @cache_control[:no_cache]