Quoting http://wiki.dlang.org/Review/std.signal :

==============================

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

==============================

As an additional comment I wonder if existing signal module should be deprecated and removed completely from Phobos even desite lack of replacement. It sits in the very same niche as this proposal and has even worse implementation with plenty of reported bugs. It becomes hard to justify it at this point.

Reply via email to