srajiv/trousers — comment on commit 4608f63b2fc7943f9cb26575e0c74982ce96beb5
File:  line 0
By shpedoikal:

This gets the job done, but it'd be easier to read and generally nicer to have a trousers struct like

struct TR_CERTIFYINFO {
union {
TPM_CERTIFY_INFO info1;
TPM_VERTIFY_INFO2 info2;
};
};

Then unpacking could be done in a function that uses the right types explicitly. As it is the code confused me a little, seeing a TPM_VERSION struct unpacked as a UINT16 made me think it was a bug.