rails commit log流し読み(2014/10/30) - なるようになるブログ

2014/10/30分のコミットです。

CHANGELOGへの追加はありませんでした。

On larger screens the email preview iframe is being limited to a height of 800 pixels, and the full available screen size is not being used.

railties/lib/rails/templates/rails/mailers/email.html.erbの修正です。

email preview画面でheight: 800px;に設定したいのを、100%に修正しています。800px以上の画面で見た際の為ですね。

Do gsub with a regexp instead of a string

gsubメソッドの第一引数にStringクラスを指定していた箇所をRegexpクラスに修正しています。

Regexpの方が早いとのこと。PRから抜粋。

require 'benchmark/ips'

# With a match: around 14% faster
Benchmark.ips do |bm|
  bm.report('regexp') { 'this is ::a random string'.gsub(/::/, '/') }
  bm.report('string') { 'this is ::a random string'.gsub('::', '/') }
  bm.compare!
end

#Calculating -------------------------------------
#              regexp     32356 i/100ms
#              string     28454 i/100ms
#-------------------------------------------------
#              regexp   480346.7 (±0.9%) i/s -    2426700 in   5.052348s
#              string   420281.9 (±1.0%) i/s -    2105596 in   5.010484s
#
#Comparison:
#              regexp:   480346.7 i/s
#              string:   420281.9 i/s - 1.14x slower


# With no match: around 33% faster
Benchmark.ips do |bm|
  bm.report('regexp') { 'this is a random string'.gsub(/::/, '/') }
  bm.report('string') { 'this is a random string'.gsub('::', '/') }
  bm.compare!
end

#Calculating -------------------------------------
#              regexp     65534 i/100ms
#              string     54729 i/100ms
#-------------------------------------------------
#              regexp  1342774.6 (±1.8%) i/s -    6750002 in   5.028622s
#              string  1012988.1 (±0.9%) i/s -    5089797 in   5.024922s
#
#Comparison:
#              regexp:  1342774.6 i/s
#              string:  1012988.1 i/s - 1.33x slower