| Both sides previous revisionPrevious revisionNext revision | Previous revision |
| rfc:hash-context.as-resource [2017/01/17 23:44] – Clarify what "reentrant" mode does pollita | rfc:hash-context.as-resource [2017/09/22 13:28] (current) – external edit 127.0.0.1 |
|---|
| * Date: 2016-12-26 | * Date: 2016-12-26 |
| * Author: Sara Golemon <pollita@php.net>, with initial implementation from Rouven Weßling <me@rouvenwessling.de> | * Author: Sara Golemon <pollita@php.net>, with initial implementation from Rouven Weßling <me@rouvenwessling.de> |
| * Status: Voting | * Status: Accepted |
| * First Published at: http://wiki.php.net/rfc/hash-context.as-resource | * First Published at: http://wiki.php.net/rfc/hash-context.as-resource |
| |
| |
| ===== Proposal ===== | ===== Proposal ===== |
| Convert the opaque resource to an opaque object. This is the lightest touch change to ensure that existing code should continue to function unless it has explciit is_resource() checks. These checks can be easily replaced with is_resource|is_object checks instead. | Convert the opaque resource to an opaque object. This is the lightest touch change to ensure that existing code should continue to function unless it has explicit is_resource() checks. These checks can be easily replaced with is_resource|is_object checks instead. |
| |
| ===== Future Scope ===== | ===== Future Scope ===== |
| This primary vote is to determine, overall, if the "hash context" resource used by hash_init/hash_update/hash_final should be changed to an object. | This primary vote is to determine, overall, if the "hash context" resource used by hash_init/hash_update/hash_final should be changed to an object. |
| |
| <doodle title="Change HashContext to an object" auth="sgolemon" voteType="single" closed="false"> | <doodle title="Change HashContext to an object" auth="sgolemon" voteType="single" closed="true"> |
| * Yes | * Yes |
| * No | * No |
| * "Reentrant": Apply both diffs in [[https://github.com/php/php-src/pull/2309|PR#2309]] which will allow contexts to continue functioning after calling hash_final() by effectively performing: $saved = hash_copy($ctx); $ret = hash_final($ctx); $ctx = $saved; | * "Reentrant": Apply both diffs in [[https://github.com/php/php-src/pull/2309|PR#2309]] which will allow contexts to continue functioning after calling hash_final() by effectively performing: $saved = hash_copy($ctx); $ret = hash_final($ctx); $ctx = $saved; |
| |
| <doodle title="Merge one or both diffs" auth="sgolemon" voteType="single" closed="false"> | <doodle title="Merge one or both diffs" auth="sgolemon" voteType="single" closed="true"> |
| * Frozen | * Frozen |
| * Reentrant | * Reentrant |