From 4381e96d74036c4a3ac5d95bab1e747d5c44f886 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sat, 31 Dec 2016 04:40:49 +0000 Subject: proxy_http_response: reduce memory pressure from larger headers We try to cleanup after ourselves if possible so future mallocs can find memory more easily. We do not need to nil the object slot, just String#clear since we're exiting the function soon, anyways. Followup to commit bb774680aae0a827f887761b18da304aa94111cc ("use String#clear for short-lived buffers we create") --- lib/yahns/proxy_http_response.rb | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/yahns/proxy_http_response.rb b/lib/yahns/proxy_http_response.rb index 1776291..7df2834 100644 --- a/lib/yahns/proxy_http_response.rb +++ b/lib/yahns/proxy_http_response.rb @@ -132,7 +132,9 @@ module Yahns::HttpResponse # :nodoc: # send the headers case rv = kgio_syssend(res, flags) - when nil then break # all done, likely + when nil # all done, likely + res.clear + break when String # partial write, highly unlikely flags = MSG_DONTWAIT res = rv # hope the skb grows -- cgit v1.2.3-24-ge0c7