Merge pull request #4722 from kennyj/fix_4708

Remove redundant instance variables (@colorize_logging)

railties/lib/rails/application/configuration.rb

@@ -112,11 +112,10 @@ def log_level
       end
 
       def colorize_logging
-        @colorize_logging
+        ActiveSupport::LogSubscriber.colorize_logging
       end
 
       def colorize_logging=(val)
-        @colorize_logging = val
         ActiveSupport::LogSubscriber.colorize_logging = val
         self.generators.colorize_logging = val
       end

railties/test/application/configuration_test.rb

@@ -534,5 +534,10 @@ def index
       assert_equal      app.env_config['action_dispatch.logger'],            Rails.logger
       assert_equal      app.env_config['action_dispatch.backtrace_cleaner'], Rails.backtrace_cleaner
     end
+
+    test "config.colorize_logging defaul is true" do
+      make_basic_app
+      assert app.config.colorize_logging
+    end
   end
 end