>> +const uint64_t *qb_get_virt_size(const QBlockStaticInfo *info)
>> +{
>> +    return info->member_addr->virt_size;
>
>Please change this to:
>
>     QBlockStaticInfoAddr addrs;
>     qb_setup_info_addr(info, &addrs);
>     return *addrs->virt_size;
>
>and similarly for the others.
>
>QBlockStaticInfoAddr should not be a public struct.  I'm sorry if this
>wasn't clear.
>
>I'll review the rest on Monday.
>
>Paolo
>
  Hi, QBlockStaticInfoAddr was declared as a pointer, user can't see
what it really is. It is actually defined in internal header files.


Reply via email to