On 2014/01/27 16:46:40, Eran wrote: > Overall LGTM, see one comment below. > I'd have ...
10 years, 3 months ago
(2014-01-28 19:03:55 UTC)
#3
On 2014/01/27 16:46:40, Eran wrote:
> Overall LGTM, see one comment below.
> I'd have liked to see the struct package being used for that, it's a bit of a
> shame we have to manually serialize variable-sized ints.
Yeah, but the "manual" option happens to result in cleaner code...
>
>
https://codereview.appspot.com/56660043/diff/20001/src/python/ct/client/tls_m...
> File src/python/ct/client/tls_message.py (right):
>
>
https://codereview.appspot.com/56660043/diff/20001/src/python/ct/client/tls_m...
> src/python/ct/client/tls_message.py:170: def get_result(self):
> Nit: Name get_serialized or serialize or something similar? Or make it
private.