>               uint64_t spa_version;
>               char *p;
>               zfs_prop_t resv_prop;
>               char *strval;
>               char msg[1024];
>  
> -             if (p = strchr(argv[0], '/'))
> +             if ((p = strchr(argv[0], '/')) != NULL)

personally, I prefer doing this as 2 lines:
p = ...;
if (p != NULL)

But I'm OK with your change too.

---
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/56/files#r49615122
_______________________________________________
developer mailing list
[email protected]
http://lists.open-zfs.org/mailman/listinfo/developer

Reply via email to