| Both sides previous revisionPrevious revisionNext revision | Previous revisionLast revisionBoth sides next revision |
| rfc:deque_straw_poll [2022/01/12 13:43] – start vote tandre | rfc:deque_straw_poll [2022/01/26 13:48] – tandre |
|---|
| * ''\Collections\Deque'' - seems like a reasonable choice of name for collections (Deque, and possible future additions such as Vector, Set, Map, Sorted Sets/Maps, etc. https://wiki.php.net/rfc/namespaces_in_bundled_extensions also allows using sub-namespaces and that may be used for things that aren't strictly collections, e.g. ''Collections\Builder\SomethingBuilder'' or functions operating on collections. | * ''\Collections\Deque'' - seems like a reasonable choice of name for collections (Deque, and possible future additions such as Vector, Set, Map, Sorted Sets/Maps, etc. https://wiki.php.net/rfc/namespaces_in_bundled_extensions also allows using sub-namespaces and that may be used for things that aren't strictly collections, e.g. ''Collections\Builder\SomethingBuilder'' or functions operating on collections. |
| |
| * ''\SplDeque'', similar to datastructures added to the Spl in PHP 5.3. \\ (I don't prefer that name because SplDoublyLinkedList, SplStack, and SplQueue are subclasses of a doubly linked list with poor performance, and this name would easily get confused with them. Also, historically, none of the functionality with that naming pattern has been final. \\ However, good documentation (e.g. suggesting *Deque instead where possible in the manual) would make that less of an issue.) \\ \\ See https://wiki.php.net/rfc/deque#lack_of_name_prefix (and arguments for https://externals.io/message/116100#116111) | * ''\SplDeque'', similar to datastructures added to the Spl in PHP 5.3. \\ (I don't prefer that name because SplDoublyLinkedList, SplStack, and SplQueue are subclasses of a doubly linked list with poor performance (accessing the offset in the middle of a linked list requires traversing half the linked list, for example), and this name would easily get confused with them. Also, historically, none of the functionality with that naming pattern has been final. \\ However, good documentation (e.g. suggesting *Deque instead where possible in the manual) would make that less of an issue.) \\ \\ See https://wiki.php.net/rfc/deque#lack_of_name_prefix (and arguments for https://externals.io/message/116100#116111) |
| |
| While there is considerable division in whether or not members of internals want to adopt namespaces, | While there is considerable division in whether or not members of internals want to adopt namespaces, |
| ===== Vote ===== | ===== Vote ===== |
| |
| Voting started on 2022-01-12 and ends on 2022-01-26. | Voting started on 2022-01-12 and ended on 2022-01-26. |
| |
| This vote will influence the name choice for the [[rfc:Deque|Deque RFC]] | This vote will influence the name choice for the [[rfc:Deque|Deque RFC]] |
| |
| First choice: | First choice: |
| <doodle title="Straw poll: Favorite choice of naming pattern" auth="tandre" voteType="single" closed="false"> | <doodle title="Straw poll: Favorite choice of naming pattern" auth="tandre" voteType="single" closed="true"> |
| * ''Deque'' | * ''Deque'' |
| * ''Collections\Deque'' | * ''Collections\Deque'' |
| Second choice: | Second choice: |
| |
| <doodle title="Straw poll: Second favorite choice of naming pattern" auth="tandre" voteType="single" closed="false"> | <doodle title="Straw poll: Second favorite choice of naming pattern" auth="tandre" voteType="single" closed="true"> |
| * ''Deque'' | * ''Deque'' |
| * ''Collections\Deque'' | * ''Collections\Deque'' |
| Third choice: | Third choice: |
| |
| <doodle title="Straw poll: Third favorite choice of naming pattern" auth="tandre" voteType="single" closed="false"> | <doodle title="Straw poll: Third favorite choice of naming pattern" auth="tandre" voteType="single" closed="true"> |
| * ''Deque'' | * ''Deque'' |
| * ''Collections\Deque'' | * ''Collections\Deque'' |