warning response not returned in gearman_client_do()

[Dennis Schön]

I've the following testcase in Gearman::XS

($ret, $result) = $client->do('warning', 'blubb');
is($ret, GEARMAN_WORK_WARNING);
is($result, 'argh');
($ret, $result) = $client->do('warning', 'blubb');
is($ret, GEARMAN_SUCCESS);
is($result, 'blubb');

the warning function in the worker is:

sub warning {
  my ($job) = @_;
  $job->send_warning("argh");
  return $job->workload();
}

with the current gearmand trunk the tests fail as the first call to gearman_client_do() returns GEARMAN_IO_WAIT instead of GEARMAN_WORK_WARNING. The change has been introduced in commit #295. I've fixed it in my branch lp:~roccoblues/gearmand/fix-warning-response

As I'm not that familiar with the gearman_client_run_tasks() function in client.c, please check if I've not broken anything else. ;-) The tests still pass.