Comment below.

On 1/8/2016 7:26 AM, ic...@apache.org wrote:
Author: icing
Date: Fri Jan  8 15:26:16 2016
New Revision: 1723737

URL: http://svn.apache.org/viewvc?rev=1723737&view=rev
Log:
normalized frame output for testing

Modified:
     httpd/test/mod_h2/trunk/mh2fuzz/h2c_conn.c
     httpd/test/mod_h2/trunk/mh2fuzz/h2c_session.c
     httpd/test/mod_h2/trunk/test/test_frames.sh

Modified: httpd/test/mod_h2/trunk/mh2fuzz/h2c_conn.c
URL: 
http://svn.apache.org/viewvc/httpd/test/mod_h2/trunk/mh2fuzz/h2c_conn.c?rev=1723737&r1=1723736&r2=1723737&view=diff
==============================================================================
--- httpd/test/mod_h2/trunk/mh2fuzz/h2c_conn.c (original)
+++ httpd/test/mod_h2/trunk/mh2fuzz/h2c_conn.c Fri Jan  8 15:26:16 2016
@@ -419,12 +419,14 @@ apr_status_t h2c_conn_shutdown(h2c_conn
              status = apr_socket_shutdown(c->socket, APR_SHUTDOWN_WRITE);
          }
- apr_socket_timeout_set(c->socket, apr_time_from_msec(250));
          h2c_conn_want_io(c, H2C_CONN_READ);
if (c->ssl) {
              c->read = tls_shutdown_read;
          }
+
+        status = c->read(c);
+        status = h2c_conn_close(c);
It seems a bit odd to me to have the status field overlaid like this with consecutive calls.
Perhaps you are planning on adding status checking code, correct?
      }
      return status;
  }



Reply via email to