From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-2.9 required=3.0 tests=ALL_TRUSTED,AWL,BAYES_00, T_RP_MATCHES_RCVD shortcircuit=no autolearn=unavailable version=3.3.2 X-Original-To: yahns-public@yhbt.net Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id 16F641F5CD for ; Fri, 13 Mar 2015 01:36:10 +0000 (UTC) From: Eric Wong To: yahns-public@yhbt.net Subject: [PATCH] avoid unnecessary calls #to_sym for __send__ Date: Fri, 13 Mar 2015 01:36:10 +0000 Message-Id: <1426210570-6833-1-git-send-email-e@80x24.org> List-Id: No point in bloating our bytecode for single-use variables. --- extras/proxy_pass.rb | 2 +- lib/yahns/rackup_handler.rb | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/extras/proxy_pass.rb b/extras/proxy_pass.rb index 08f9730..15cbae5 100644 --- a/extras/proxy_pass.rb +++ b/extras/proxy_pass.rb @@ -134,7 +134,7 @@ class ProxyPass # :nodoc: def call(env) request_method = env['REQUEST_METHOD'] req = Rack::Request.new(env) - path = @path.gsub(/\$(\w+)/) { req.__send__($1.to_sym) } + path = @path.gsub(/\$(\w+)/) { req.__send__($1) } req = "#{request_method} #{path} HTTP/1.1\r\n" \ "X-Forwarded-For: #{env["REMOTE_ADDR"]}\r\n" diff --git a/lib/yahns/rackup_handler.rb b/lib/yahns/rackup_handler.rb index 127db48..3d7beab 100644 --- a/lib/yahns/rackup_handler.rb +++ b/lib/yahns/rackup_handler.rb @@ -32,7 +32,6 @@ module Yahns::RackupHandler # :nodoc: end %w(stderr_path stdout_path).each do |x| - x = x.to_sym val = o[x] and __send__(x, val) end end -- EW