diff options
author | Eric Wong <normalperson@yhbt.net> | 2009-08-02 13:51:20 -0700 |
---|---|---|
committer | Eric Wong <normalperson@yhbt.net> | 2009-08-09 01:24:46 -0700 |
commit | 4dc2a9e95d115547b7c2680f198293d6e25658b3 (patch) | |
tree | c61a7c69368e52ecc81efe87d35409ab1fb9931e /ext/unicorn_http/unicorn_http.rl | |
parent | ca3fd8acb4bc3f5125a9f0f281951fb8f03778e5 (diff) | |
download | unicorn-4dc2a9e95d115547b7c2680f198293d6e25658b3.tar.gz |
http: find_common_field_value => find_common_field
The "_value" suffix was used to denote the return type, which is redundandant as it is already known at compile time (being that this is C and functions have explicit return types). Furthurmore, the "_value" is confusing since we're actually returning a "key" when "value" is used in the context of "key-value" pairs.
Diffstat (limited to 'ext/unicorn_http/unicorn_http.rl')
-rw-r--r-- | ext/unicorn_http/unicorn_http.rl | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ext/unicorn_http/unicorn_http.rl b/ext/unicorn_http/unicorn_http.rl index 86ef46e..706c77c 100644 --- a/ext/unicorn_http/unicorn_http.rl +++ b/ext/unicorn_http/unicorn_http.rl @@ -148,7 +148,7 @@ static void http_field(VALUE req, const char *field, VALIDATE_MAX_LENGTH(flen, FIELD_NAME); VALIDATE_MAX_LENGTH(vlen, FIELD_VALUE); - f = find_common_field_value(field, flen); + f = find_common_field(field, flen); if (f == Qnil) { /* @@ -386,6 +386,6 @@ void Init_unicorn_http(void) rb_define_method(cHttpParser, "execute", HttpParser_execute,2); sym_http_body = ID2SYM(rb_intern("http_body")); init_common_fields(); - g_http_host = find_common_field_value("HOST", 4); + g_http_host = find_common_field("HOST", 4); assert(g_http_host != Qnil); } |