typo
authorMichael Wallner <mike@php.net>
Mon, 19 Jan 2015 10:52:13 +0000 (11:52 +0100)
committerMichael Wallner <mike@php.net>
Mon, 19 Jan 2015 10:52:13 +0000 (11:52 +0100)
php_http_env.c
php_http_negotiate.c
tests/negotiate001.phpt

index a2c3227ea94127d85a2b6b3dfc3bec3dcc8c8d3d..239af9812a25141cfbfc1885f05150315d41b8bd 100644 (file)
@@ -137,7 +137,7 @@ char *php_http_env_get_request_header(const char *name_str, size_t name_len, siz
                request_headers = PHP_HTTP_G->env.request.headers;
        }
 
-       if ((zvalue == zend_symtable_str_find(request_headers, key, name_len))) {
+       if ((zvalue = zend_symtable_str_find(request_headers, key, name_len))) {
                zend_string *zs = zval_get_string(zvalue);
 
                val = estrndup(zs->val, zs->len);
index f0526dc5a1bd9c3a5ab9899471dd7184b730dbab..1535b86c67f1c077cb49cb8337b0e99113ea5a3d 100644 (file)
@@ -27,10 +27,10 @@ static int php_http_negotiate_sort(const void *first, const void *second TSRMLS_
 #define M_SEC 2
 #define M_ANY 1
 #define M_NOT 0
-#define M_ALL -1
+#define M_ALL ~0
 static inline unsigned php_http_negotiate_match(const char *param_str, size_t param_len, const char *supported_str, size_t supported_len, const char *sep_str, size_t sep_len)
 {
-       int match = M_NOT;
+       unsigned match = M_NOT;
 
        if (param_len == supported_len && !strncasecmp(param_str, supported_str, param_len)) {
                /* that was easy */
@@ -131,6 +131,10 @@ HashTable *php_http_negotiate(const char *value_str, size_t value_len, HashTable
                                q = 1.0 - ++i / 100.0;
                        }
 
+#if 0
+                       fprintf(stderr, "Q: %s=%1.3f\n", key.key->val, q);
+#endif
+
                        if (key.key) {
                                add_assoc_double_ex(&arr, key.key->val, key.key->len, q);
                        } else {
index f628cdf1d9944784da44594d282bd04857f3549a..0e553f50154e418a2e50531d1ceb91e0e5a9a670 100644 (file)
@@ -122,7 +122,7 @@ CUSTOM
 a.b: Array
 (
     [a.b] => 0.9
-    [c.e] => 0.1
     [a.x] => 0.1
+    [c.e] => 0.1
 )
 DONE