diff options
| author | utilum <oz@utilum.com> | 2019-02-01 00:46:11 +0100 |
|---|---|---|
| committer | utilum <oz@utilum.com> | 2019-02-13 02:11:21 +0100 |
| commit | 87a5379b422d38ea8766af1537593af44b6dffc6 (patch) | |
| tree | f77e757600a1ef91ab5a21cfa955036ece8c1fb6 /activesupport | |
| parent | 8d9d46fdae4502b8881eca641c51a64ee7804194 (diff) | |
| download | rails-87a5379b422d38ea8766af1537593af44b6dffc6.tar.gz rails-87a5379b422d38ea8766af1537593af44b6dffc6.tar.bz2 rails-87a5379b422d38ea8766af1537593af44b6dffc6.zip | |
Ruby 2.7 warning: creating a Proc without a block
As of [Revision 66772](
https://bugs.ruby-lang.org/projects/ruby-trunk/repository/trunk/revisions/66772)
`Proc.new` without giving a block emits `warning: tried to create Proc object without a block`.
This commit fixes cases where Rails test suit tickles this warning.
See CI logs:
https://travis-ci.org/rails/rails/jobs/487205819#L1161-L1190
https://travis-ci.org/rails/rails/jobs/487205821#L1154-1159
https://travis-ci.org/rails/rails/jobs/487205821#L1160-L1169
https://travis-ci.org/rails/rails/jobs/487205821#L1189
https://travis-ci.org/rails/rails/jobs/487254404#L1307-L1416
https://travis-ci.org/rails/rails/jobs/487254405#L1174-L1191
Diffstat (limited to 'activesupport')
| -rw-r--r-- | activesupport/lib/active_support/notifications/fanout.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/activesupport/lib/active_support/notifications/fanout.rb b/activesupport/lib/active_support/notifications/fanout.rb index f06504cf2c..c506b35b1e 100644 --- a/activesupport/lib/active_support/notifications/fanout.rb +++ b/activesupport/lib/active_support/notifications/fanout.rb @@ -20,8 +20,8 @@ module ActiveSupport super end - def subscribe(pattern = nil, block = Proc.new) - subscriber = Subscribers.new pattern, block + def subscribe(pattern = nil, callable = nil, &block) + subscriber = Subscribers.new(pattern, callable || block) synchronize do if String === pattern @string_subscribers[pattern] << subscriber |