rfc:additional-context-in-pcntl-signal-handler
Differences
This shows you the differences between two versions of the page.
Both sides previous revisionPrevious revisionNext revision | Previous revision | ||
rfc:additional-context-in-pcntl-signal-handler [2016/06/23 17:18] – bishop | rfc:additional-context-in-pcntl-signal-handler [2017/09/22 13:28] (current) – external edit 127.0.0.1 | ||
---|---|---|---|
Line 1: | Line 1: | ||
====== PHP RFC: Additional Context in pcntl_signal Handler ====== | ====== PHP RFC: Additional Context in pcntl_signal Handler ====== | ||
- | * Version: | + | * Version: |
* Date: 2016-06-14 | * Date: 2016-06-14 | ||
* Authors: David Walker < | * Authors: David Walker < | ||
- | * Status: | + | * Status: |
* First Published at: http:// | * First Published at: http:// | ||
Line 52: | Line 52: | ||
===== Proposed PHP Version(s) ===== | ===== Proposed PHP Version(s) ===== | ||
- | Next PHP 7.x. | + | Next PHP 7.x, currently 7.1. |
Line 70: | Line 70: | ||
None. | None. | ||
+ | ===== Discussion ===== | ||
+ | |||
+ | Instead of updating '' | ||
+ | |||
+ | Performance was raised as a concern. Having run tests through callgrind there is an additional 0.0001% cost for the feature. The profiled code defined an empty function, set the handler, and triggered the signal. Passing the additional information resulted in 2000 extra instructions, | ||
===== Open Issues ===== | ===== Open Issues ===== | ||
Line 81: | Line 86: | ||
Requires a 50%+1 majority. | Requires a 50%+1 majority. | ||
+ | <doodle title=" | ||
+ | * Yes | ||
+ | * No | ||
+ | </ | ||
+ | |||
+ | Voting shall close one week after opening on Thursday, July 14, 2016 at 23:59 UTC. | ||
===== Patches and Tests ===== | ===== Patches and Tests ===== | ||
- | * [[https:// | + | * [[https:// |
Line 90: | Line 101: | ||
* [[https:// | * [[https:// | ||
+ | * [[https:// |
rfc/additional-context-in-pcntl-signal-handler.txt · Last modified: 2017/09/22 13:28 by 127.0.0.1