yahns Ruby server user/dev discussion
 help / color / mirror / code / Atom feed
Search results ordered by [date|relevance]  view[summary|nested|Atom feed]
thread overview below | download mbox.gz: |
* [PATCH] avoid unnecessary calls #to_sym for __send__
@ 2015-03-13  1:36  7% Eric Wong
  0 siblings, 0 replies; 1+ results
From: Eric Wong @ 2015-03-13  1:36 UTC (permalink / raw)
  To: yahns-public

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


^ permalink raw reply related	[relevance 7%]

Results 1-1 of 1 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2015-03-13  1:36  7% [PATCH] avoid unnecessary calls #to_sym for __send__ Eric Wong

Code repositories for project(s) associated with this public inbox

	https://yhbt.net/yahns.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).