comment on

Hi, Monks

I have some multithreaded code which gives me warnings at the end:

Perl exited with active threads: 1 running and unjoined 2 finished and unjoined 0 running and detached

my $q  = Thread::Queue->new();
my $pq = Thread::Queue->new();

my @threads =
  map { threads->create( \&worker, $_ ) } ( 1 .. $config->{number_of_t
+hreads} );

push @threads, threads->create( \&controller );

my $mech = WWW::Mechanize->new();
$mech->agent_alias("Windows IE 6");

$mech->get( $config->{start_url} );

my @states = find_states($mech);

foreach my $state (@states) {

    process_state($state);
}

while ( $q->pending() ) {

    sleep 1;
}

foreach ( 1 .. $config->{number_of_threads} ) {

    $q->enqueue(undef);
}

while ( $pq->pending() ) {

    sleep 1;
}

$pq->enqueue(undef);

foreach my $thr (@threads) {

    if ( $thr->is_joinable() ) {
        $thr->join();
    }
}
[download]

sub process_state() is populating $q queue. Each thread contain something like this:

    while ( my $org = $q->dequeue() ) {

        parse_org( $org, $mech );
        $pq->enqueue($org);
    }
[download]

Thanks. Roman

In reply to What is the correct way to finish multithreaded program? by Gangabass

Are you posting in the right place? Check out Where do I post X? to know for sure.
Posts may use any of the Perl Monks Approved HTML tags. Currently these include the following:
<code> <a> <b> <big> <blockquote> <br /> <dd> <dl> <dt> <em> <font> <h1> <h2> <h3> <h4> <h5> <h6> <hr /> <i> <li> <nbsp> <ol> <p> <small> <strike> <strong> <sub> <sup> <table> <td> <th> <tr> <tt> <u> <ul>
Snippets of code should be wrapped in <code> tags not <pre> tags. In fact, <pre> tags should generally be avoided. If they must be used, extreme care should be taken to ensure that their contents do not have long lines (<70 chars), in order to prevent horizontal scrolling (and possible janitor intervention).
Want more info? How to link or How to display code and escape characters are good places to start.


laziness, impatience, and hubris
	PerlMonks