Wei Yang <[email protected]> wrote:
> On Wed, May 15, 2019 at 02:15:41PM +0200, Juan Quintela wrote:
>>Signed-off-by: Juan Quintela <[email protected]>
>>@@ -1821,6 +1826,18 @@ void hmp_migrate_set_parameter(Monitor *mon, const 
>>QDict *qdict)
>>         p->has_multifd_channels = true;
>>         visit_type_int(v, param, &p->multifd_channels, &err);
>>         break;
>>+    case MIGRATION_PARAMETER_MULTIFD_COMPRESS:
>>+        p->has_multifd_compress = true;
>>+        visit_type_MultifdCompress(v, param, &compress_type, &err);
>>+        if (err) {
>>+            break;
>>+        }
>>+        if (compress_type < 0 || compress_type >= MULTIFD_COMPRESS__MAX) {
>
> A warning during build:
>
> hmp.c:1835:27: warning: comparison of unsigned enum expression < 0 is
> always false [-Wtautological-compare]
>         if (compress_type < 0 || compress_type >= MULTIFD_COMPRESS__MAX) {

Fixed, see Markus reason for dropping this bit.

/me wonders why this didn't failed my compilation ....

Thanks, Juan.

Reply via email to