Skip to content
This repository

add require to suppress warning; remove variable #10080

Merged
merged 1 commit into from about 1 year ago

2 participants

Vipul A M Steve Klabnik
Vipul A M
  1. add require to expose to_d method and suppress warning
  2. drop unnecessary variable assignment

Not so sure about 1. but wanted to propose the change

Steve Klabnik steveklabnik merged commit 81ed4f5 into from April 03, 2013
Steve Klabnik steveklabnik closed this April 03, 2013
Steve Klabnik
Collaborator

Nice! I was investigating the to_d issue, this is nicer.

Vipul A M vipulnsward deleted the branch April 03, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 1 unique commit by 1 author.

Apr 04, 2013
Vipul A M add require to suppress warning; remove variable 4da888f
This page is out of date. Refresh to see the latest.
1  activesupport/lib/active_support/core_ext/big_decimal/conversions.rb
... ...
@@ -1,4 +1,5 @@
1 1
 require 'bigdecimal'
  2
+require 'bigdecimal/util'
2 3
 require 'yaml'
3 4
 
4 5
 class BigDecimal
2  railties/lib/rails/generators/named_base.rb
@@ -40,7 +40,7 @@ def module_namespacing(&block)
40 40
 
41 41
         def indent(content, multiplier = 2)
42 42
           spaces = " " * multiplier
43  
-          content = content.each_line.map {|line| line.blank? ? line : "#{spaces}#{line}" }.join
  43
+          content.each_line.map {|line| line.blank? ? line : "#{spaces}#{line}" }.join
44 44
         end
45 45
 
46 46
         def wrap_with_namespace(content)
Commit_comment_tip

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.