why LinkedBlockingQueue's unlink() not need do notEmpty.signal() just like poll() ?

LinkedBlockingQueue's unlink() is as follows:
void unlink(Node<E> p, Node<E> trail) {
// assert isFullyLocked();
    // p.next is not changed, to allow iterators that are
    // traversing p to maintain their weak-consistency guarantee.
p.item = null;
trail.next = p.next;
    if (last == p)
last = trail;
    if (count.getAndDecrement() == capacity)
notFull.signal();
}


unlink() will be invoked by remove(Object o).


Why LinkedBlockingQueue's unlink() not need do notEmpty.signal() just like poll() ?


Maybe it could be like this:
    void unlink(Node<E> p, Node<E> trail) {
        // assert isFullyLocked();
        // p.next is not changed, to allow iterators that are
        // traversing p to maintain their weak-consistency guarantee.
        p.item = null;
        trail.next = p.next;
        if (last == p)
            last = trail;
        int c = count.getAndDecrement();
        if (c > 1)
            notEmpty.signal();
        if (c == capacity)
            notFull.signal();
    }


PS:JDK8.


--------------------------------------------------------------------------------
Regards
Liu

_______________________________________________
Concurrency-interest mailing list
[email protected]
http://cs.oswego.edu/mailman/listinfo/concurrency-interest