Skip to content

Commit c85a918

Browse files
committed
clarifying comment
1 parent 226a739 commit c85a918

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

‎sapi/cli/php_cli_server.c‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1712,8 +1712,9 @@ static void php_cli_server_client_save_header(php_cli_server_client *client)
17121712

17131713
/* Add/Update the wrapped zend_string to the HashTable */
17141714
zend_hash_update(&client->request.headers, lc_header_name, &tmp);
1715-
/* A repeat spelled differently leaves the first spelling's entry in place while the
1716-
* update above releases its value, so this table must own a reference too. */
1715+
/* Each table owns a reference to the new value. The old value can outlive the update
1716+
* above, as a repeat spelled differently keeps the first spelling's entry in
1717+
* headers_original_case. */
17171718
zend_string_addref(Z_STR(tmp));
17181719
zend_hash_update(&client->request.headers_original_case, client->current_header_name, &tmp);
17191720

0 commit comments

Comments
 (0)