View source for Review/std.signal

== Description ==

[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 ===

[https://vhios.dyndns.org/dlang.org/web/phobos/std_signal.html Documentation]

== 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.