Difference between revisions of "Review/std.signal"
(std.signal page template) |
(std.signal voting results) |
||
(6 intermediate revisions by 2 users not shown) | |||
Line 1: | Line 1: | ||
== Description == | == Description == | ||
− | [https://github.com/phobos-x/phobosx/blob/master/source/phobosx/signal.d std. | + | [https://github.com/phobos-x/phobosx/blob/master/source/phobosx/signal.d std.signal] is a package authored by Robert Klotzner that implements [http://en.wikipedia.org/wiki/Observer_pattern Observer pattern] in D. It is supposed to deprecate and supersede existing (and broken) std.signals |
=== Related links === | === Related links === | ||
Line 8: | Line 8: | ||
== Current state == | == Current state == | ||
− | Review | + | |
+ | Failed voting. Encouraged to stay in semi-official state until gets more recognition / maturity. | ||
+ | |||
+ | == Review 1 == | ||
+ | |||
+ | === Review thread === | ||
+ | |||
+ | http://forum.dlang.org/post/ujlhznaphepibgtpcoqz@forum.dlang.org | ||
+ | |||
+ | === Outcome === | ||
+ | |||
+ | Review thread did not get much attention. Several relatively minor concerns have been raised and fixed. In absence of any additional feedback proposal has been moved to voting stage. | ||
+ | |||
+ | == Voting == | ||
+ | |||
+ | === Voting thread === | ||
+ | |||
+ | http://forum.dlang.org/post/eeenynxifropasqcufdg@forum.dlang.org | ||
+ | |||
+ | === Stats === | ||
+ | |||
+ | ==== Yes ==== | ||
+ | * ilya-stromberg | ||
+ | * Damian Day | ||
+ | * Robert M. Münch | ||
+ | |||
+ | ==== No ==== | ||
+ | * Jakob Ovrum | ||
+ | * Andrei Alexandrescu * | ||
+ | * Denis Shelomovskij | ||
+ | |||
+ | People marked with asterisk belong to this list : https://github.com/D-Programming-Language?tab=members | ||
+ | |||
+ | === Summary === | ||
+ | |||
+ | Total : 6 | ||
+ | Yes : 3 | ||
+ | Yes, If : 0 | ||
+ | No : 3 | ||
+ | |||
+ | Among D developers : 0/0/1 | ||
+ | |||
+ | === Verdict === | ||
+ | |||
+ | Rejected in its current implementation | ||
+ | |||
+ | === Review manager comment === | ||
+ | |||
+ | Apart from some implementation / documentation issues mentioned by reviewers important concern is general lack of interest in such module. It looks too niche to get amount of attention needed for detailed reviewing and this will remain concern even if all issues will be addressed. One possible recommendation is to keep module as separate dub package in Phobos candidate category so that it can't be easily discovered by those seeking sinal implementation and go for another review once it gets some user base to back it. |
Latest revision as of 09:17, 21 January 2014
Contents
Description
std.signal is a package authored by Robert Klotzner that implements Observer pattern in D. It is supposed to deprecate and supersede existing (and broken) std.signals
Related links
Current state
Failed voting. Encouraged to stay in semi-official state until gets more recognition / maturity.
Review 1
Review thread
http://forum.dlang.org/post/ujlhznaphepibgtpcoqz@forum.dlang.org
Outcome
Review thread did not get much attention. Several relatively minor concerns have been raised and fixed. In absence of any additional feedback proposal has been moved to voting stage.
Voting
Voting thread
http://forum.dlang.org/post/eeenynxifropasqcufdg@forum.dlang.org
Stats
Yes
- ilya-stromberg
- Damian Day
- Robert M. Münch
No
- Jakob Ovrum
- Andrei Alexandrescu *
- Denis Shelomovskij
People marked with asterisk belong to this list : https://github.com/D-Programming-Language?tab=members
Summary
Total : 6 Yes : 3 Yes, If : 0 No : 3
Among D developers : 0/0/1
Verdict
Rejected in its current implementation
Review manager comment
Apart from some implementation / documentation issues mentioned by reviewers important concern is general lack of interest in such module. It looks too niche to get amount of attention needed for detailed reviewing and this will remain concern even if all issues will be addressed. One possible recommendation is to keep module as separate dub package in Phobos candidate category so that it can't be easily discovered by those seeking sinal implementation and go for another review once it gets some user base to back it.