Fixed a logic error with my recent gzip patch #91

Merged
merged 1 commit into from Feb 22, 2013

Projects

None yet

2 participants

@comptonqc

The gzip patch I submitted recently assumes that apr_socket_data_get returns with an error code if there is no data in the socket; however, this is not always the case. This patch explicitly checks for the data we want instead of relying on the error code to indicate its presence.

@comptonqc comptonqc Fixed a logic error with my recent gzip patch
The gzip patch I submitted recently assumes that apr_socket_data_get
returns with an error code if there is no data in the socket;
however, this is not always the case. This patch explicitly checks
for the data we want instead of relying on the error code to
indicate its presence.
05f0453
@jbuchbinder jbuchbinder merged commit 0ae75b6 into ganglia:master Feb 22, 2013

1 check passed

Details default The Travis build passed
@comptonqc comptonqc deleted the comptonqc:patch-gzip_fix2 branch Feb 22, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment