Patch Set 2 LGTM Nits only https://codereview.appspot.com/291170043/diff/20001/lib/ssl/sslimpl.h File lib/ssl/sslimpl.h (right): https://codereview.appspot.com/291170043/diff/20001/lib/ssl/sslimpl.h#newcode89 lib/ssl/sslimpl.h:89: #define PRINT_KEY(a, b) if (ssl_trace >= (a)) ssl_PrintKey b Nit: wrap like the lines above, since that is what clang-format will end up doing. Though maybe the indent there can be fixed. https://codereview.appspot.com/291170043/diff/20001/lib/ssl/tls13con.c File lib/ssl/tls13con.c (right): https://codereview.appspot.com/291170043/diff/20001/lib/ssl/tls13con.c#newcod... lib/ssl/tls13con.c:1275: extra blank line https://codereview.appspot.com/291170043/diff/20001/lib/ssl/tls13con.c#newcod... lib/ssl/tls13con.c:1277: PRINT_BUF(90, (NULL, "Context string", context_string, strlen(context_string))); Why omit the socket argument here? You have one. https://codereview.appspot.com/291170043/diff/20001/lib/ssl/tls13hkdf.c File lib/ssl/tls13hkdf.c (right): https://codereview.appspot.com/291170043/diff/20001/lib/ssl/tls13hkdf.c#newco... lib/ssl/tls13hkdf.c:174: SSL_TRC(60, ("HKDF Expand: label=[TLS 1.3, ] + '%s',requested length=%d", Why not just "... label='TLS 1.3, %s',requested ..."
https://codereview.appspot.com/291170043/diff/20001/lib/ssl/sslimpl.h File lib/ssl/sslimpl.h (right): https://codereview.appspot.com/291170043/diff/20001/lib/ssl/sslimpl.h#newcode89 lib/ssl/sslimpl.h:89: #define PRINT_KEY(a, b) if (ssl_trace >= (a)) ssl_PrintKey b On 2016/03/14 00:14:17, mt wrote: > Nit: wrap like the lines above, since that is what clang-format will end up > doing. Though maybe the indent there can be fixed. Done. https://codereview.appspot.com/291170043/diff/20001/lib/ssl/tls13con.c File lib/ssl/tls13con.c (right): https://codereview.appspot.com/291170043/diff/20001/lib/ssl/tls13con.c#newcod... lib/ssl/tls13con.c:1275: On 2016/03/14 00:14:18, mt wrote: > extra blank line Done. https://codereview.appspot.com/291170043/diff/20001/lib/ssl/tls13con.c#newcod... lib/ssl/tls13con.c:1277: PRINT_BUF(90, (NULL, "Context string", context_string, strlen(context_string))); On 2016/03/14 00:14:18, mt wrote: > Why omit the socket argument here? You have one. https://codereview.appspot.com/291170043/diff/20001/lib/ssl/tls13hkdf.c File lib/ssl/tls13hkdf.c (right): https://codereview.appspot.com/291170043/diff/20001/lib/ssl/tls13hkdf.c#newco... lib/ssl/tls13hkdf.c:174: SSL_TRC(60, ("HKDF Expand: label=[TLS 1.3, ] + '%s',requested length=%d", On 2016/03/14 00:14:18, mt wrote: > Why not just "... label='TLS 1.3, %s',requested ..." Because I wanted to indicate the data that I wasactually using here. And the TLS 1.3 is inserted separately