From: tenderlove@... Date: 2014-03-12T18:17:07+00:00 Subject: [ruby-core:61440] [ruby-trunk - Bug #9593] Keyword arguments default argument assignment behaviour not consistent with optional argument Issue #9593 has been updated by Aaron Patterson. Hi, Before r45272 this code worked: ~~~ require 'minitest/autorun' class DefaultArg < MiniTest::Unit::TestCase class Default def foo; 'foo'; end def bar foo = foo foo end end def test_default assert_equal 'foo', Default.new.bar end end ~~~ Was this expected to break? Unfortunately we have real code that depends on the behavior: https://github.com/rails/rails/blob/81d08abcccf2ef1b0ea6e98daf00f6ca375f3d8a/activerecord/lib/active_record/associations/has_many_association.rb#L74 but we can change it ---------------------------------------- Bug #9593: Keyword arguments default argument assignment behaviour not consistent with optional argument https://bugs.ruby-lang.org/issues/9593#change-45742 * Author: Jack Chen * Status: Closed * Priority: Normal * Assignee: Yukihiro Matsumoto * Category: syntax * Target version: current: 2.2.0 * ruby -v: 2.1.1 * Backport: 1.9.3: REQUIRED, 2.0.0: REQUIRED, 2.1: REQUIRED ---------------------------------------- Given the following code: def var 100 end def foo(var: var + 1) puts "var: #{var.inspect}" end def bar(var = var + 1) puts "var: #{var.inspect}" end foo(var: 1) foo rescue p $! bar(1) bar Ruby 2.0.0: var: 1 var: 101 var: 1 var: 101 Ruby 2.1.1: var: 1 # var: 1 var: 101 What appears to be happening is that since 2.1.1, the keyword argument defines `var` as a variable before evaluating the default argument. Personally, I prefer 2.0.0 behaviour, but the way 2.1.1 handles default arguments in non keyword arguments is inconsistent. -- http://bugs.ruby-lang.org/