diff options
author | Pat Allan <pat@freelancing-gods.com> | 2017-05-21 14:01:30 +1000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2017-05-21 04:18:46 +0000 |
commit | e10ef6433f47f6152b776237c5f408e35c186dce (patch) | |
tree | a30271a2bec96ef6b00347d47ba0e7a28721e391 /lib/clogger | |
parent | 029d072d420f0adf98c620913fe05eba3222e244 (diff) | |
download | clogger-e10ef6433f47f6152b776237c5f408e35c186dce.tar.gz |
Rack (since v2) has started explicitly listing the second (optional) argument for respond_to?, which matches the underlying Ruby spec. This patch fixes the calls in both C and Ruby approaches. However, rb_respond_to only accepts a single argument - differing from the Ruby side of things - so perhaps this patch isn't quite perfect (and my C skills are very limited, so the whole thing could use a review).
Diffstat (limited to 'lib/clogger')
-rw-r--r-- | lib/clogger/pure.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/clogger/pure.rb b/lib/clogger/pure.rb index 77f81b4..fddfe79 100644 --- a/lib/clogger/pure.rb +++ b/lib/clogger/pure.rb @@ -77,8 +77,8 @@ class Clogger @logger.respond_to?(:fileno) ? @logger.fileno : nil end - def respond_to?(m) - :close == m.to_sym || @body.respond_to?(m) + def respond_to?(method, include_all=false) + :close == method.to_sym || @body.respond_to?(method, include_all) end def to_path |