search

p-alik / perl-Gearman

9 stars 10 forks source link

fix: Use given timeout value for failure reason #36

Closed hidden-primary-net closed 6 years ago

hidden-primary-net commented 6 years ago

My fix for issue #33 introduced a timeout failure reason as argument for the on_fail callback. This reason was intended to contain the given timeout value.

The timeout value is changed to contain the current timestamp rendering this value useless for the message.

This patch introduces another variable storing the given value and uses this new value for the reason.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.06%) to 86.667% when pulling 8e5c0aaea77bafa96e4915cb40df309bf2932ac0 on hidden-primary-net:invalid_timeout_reason_issue35 into e063807e27a6e136ad0ef0b91b7c2634500cadcc on p-alik:master.

p-alik commented 6 years ago

thank you, Marco.