Skip Menu |

This queue is for tickets about the Thread-Queue CPAN distribution.

Report information
The Basics
Id: 107461
Status: resolved
Priority: 0/
Queue: Thread-Queue
People
Owner: Nobody in particular
Requestors: karlroyer [...] corp.cardboardfish.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 3.07

History Show all quoted text
  Fri Oct 02 06:51:25 2015 karlroyer [...] corp.cardboardfish.com - Ticket created
Subject: The limit option blocks.
Date: Fri, 2 Oct 2015 11:50:51 +0100
To: bug-Thread-Queue [...] rt.cpan.org
From: Karl Royer <karlroyer [...] corp.cardboardfish.com>
If the limit is x and the queue has x items, calling dequeue(x) does not signal to blocked threads enque'ing. The dequeue function needs:- cond_signal(%$self) if ((@$queue > $count) || $$self{'ENDED'}); changed to:- cond_signal(%$self) if ((@$queue >= $count) || $$self{'ENDED'}); else things get a little stuck!.
  Tue Oct 27 15:33:43 2015 jdhedden [...] cpan.org - Correspondence added
Thanks for this bug report. A fix has been applied, and a new version of Thread::Queue has been upload to CPAN. On 2015-10-02 06:51:25, karlroyer@corp.cardboardfish.com wrote: Show quoted text
> If the limit is x and the queue has x items, calling dequeue(x) does not > signal to blocked threads enque'ing. > > The dequeue function needs:- > > cond_signal(%$self) if ((@$queue > $count) || $$self{'ENDED'}); > > changed to:- > > cond_signal(%$self) if ((@$queue >= $count) || $$self{'ENDED'}); > > else things get a little stuck!.
  Tue Oct 27 15:33:44 2015 The RT System itself - Status changed from 'new' to 'open'
  Tue Oct 27 15:33:47 2015 jdhedden [...] cpan.org - Status changed from 'open' to 'resolved'
  Tue Oct 27 15:33:47 2015 jdhedden [...] cpan.org - Fixed in 3.07 added