> +/**
> + * enum sgx_encls_leaves - return codes for ENCLS, ENCLU and ENCLV
> + * %SGX_SUCCESS: No error.
> + * %SGX_INVALID_SIG_STRUCT: SIGSTRUCT contains an invalid value.
> + * %SGX_INVALID_ATTRIBUTE: Enclave is not attempting to access a resource
> + * for which it is not authorized.
> + * %SGX_BLKSTATE: EPC page is already blocked.
> + * %SGX_INVALID_MEASUREMENT: SIGSTRUCT or EINITTOKEN contains an incorrect
> + * measurement.
...
> +enum sgx_return_codes {
> + SGX_SUCCESS = 0,
> + SGX_INVALID_SIG_STRUCT = 1,
> + SGX_INVALID_ATTRIBUTE = 2,
> + SGX_BLKSTATE = 3,
> + SGX_INVALID_MEASUREMENT = 4,
...
I don't think I've ever seen this particular method of commenting
before. It's rather verbose and duplicates the names twice, which seems
a bit silly.
Can you talk a bit about why you chose to do it this way? I'd
personally much rather see at least some brief comments inline with the
definitions.
