rfc:include_cleanup
Differences
This shows you the differences between two versions of the page.
Both sides previous revisionPrevious revisionNext revision | Previous revision | ||
rfc:include_cleanup [2023/01/30 09:23] – additions to "Criticism" section, added PR links maxk | rfc:include_cleanup [2023/02/15 15:11] (current) – declined maxk | ||
---|---|---|---|
Line 3: | Line 3: | ||
* Date: 2023-01-18 | * Date: 2023-01-18 | ||
* Author: Max Kellermann, max.kellermann@ionos.com | * Author: Max Kellermann, max.kellermann@ionos.com | ||
- | * Status: | + | * Status: |
* First Published at: https:// | * First Published at: https:// | ||
Line 105: | Line 105: | ||
* Is it allowed to forward-declare structs/ | * Is it allowed to forward-declare structs/ | ||
* Is it allowed to split a large header, e.g. move zend_result to a separate header, to reduce dependencies on catch-all headers such as zend_types.h? | * Is it allowed to split a large header, e.g. move zend_result to a separate header, to reduce dependencies on catch-all headers such as zend_types.h? | ||
+ | |||
+ | ===== Vote ===== | ||
+ | |||
+ | Voting started on 2023-02-01 (two weeks after the RFC was posted to php-internals) and will end on 2023-02-15 at 15:00 UTC (another two weeks). | ||
+ | |||
+ | Primary vote: | ||
+ | |||
+ | <doodle title=" | ||
+ | * Yes | ||
+ | * No | ||
+ | </ | ||
+ | |||
+ | Secondary (clarifications on how to clean up): | ||
+ | |||
+ | <doodle title=" | ||
+ | * Yes | ||
+ | * No | ||
+ | </ | ||
+ | |||
+ | <doodle title=" | ||
+ | * Yes | ||
+ | * No | ||
+ | </ | ||
+ | |||
+ | <doodle title=" | ||
+ | * Yes | ||
+ | * No | ||
+ | </ | ||
===== Patches and Tests ===== | ===== Patches and Tests ===== | ||
Line 115: | Line 143: | ||
https:// | https:// | ||
+ | |||
+ | If voters decide that #include directives should not have code comments, then existing comments should be removed to reduce " | ||
===== References ===== | ===== References ===== |
rfc/include_cleanup.1675070627.txt.gz · Last modified: 2023/01/30 09:23 by maxk