"Dr. David Alan Gilbert" <dgilb...@redhat.com> wrote:
> * Juan Quintela (quint...@redhat.com) wrote:
>> It will be used later.
>>
>> Signed-off-by: Juan Quintela <quint...@redhat.com>
>> + */
>> +static int nocomp_recv_pages(MultiFDRecvParams *p, uint32_t used, Error
>> **errp)
>> +{
>> + if (p->flags != 0) {
>> + error_setg(errp, "multifd %d: flags received %x flags expected %x",
>> + p->id, MULTIFD_FLAG_ZLIB, p->flags);
>
> Can you just explain that a bit - the 'received' seems to be constant
> while the expected is p->flags - is that the right way around?
> Why would you expect FLAG_ZLIB in nocomp?
When I changed printf's to error_setg I did a bit *too much* of copy
paste. Then I decided to put consistently received/expected in all
messages and clearly I failed.
Fixing, Thanks.
