Difference between revisions of "Review/std.signal"

From D Wiki
Jump to: navigation, search
m (Description)
(std.signal voting results)
 
(One intermediate revision by the same user not shown)
Line 7: Line 7:
 
[https://vhios.dyndns.org/dlang.org/web/phobos/std_signal.html Documentation]
 
[https://vhios.dyndns.org/dlang.org/web/phobos/std_signal.html Documentation]
  
[http://forum.dlang.org/post/ujlhznaphepibgtpcoqz@forum.dlang.org Review thread 1]
+
== Current state ==
 +
 
 +
Failed voting. Encouraged to stay in semi-official state until gets more recognition / maturity.
 +
 
 +
== Review 1 ==
 +
 
 +
=== Review thread ===
  
== Current state ==
+
http://forum.dlang.org/post/ujlhznaphepibgtpcoqz@forum.dlang.org
Under voting (http://forum.dlang.org/post/nhnitsaqijyhnmgcumux@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

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

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.