----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/181/#review609 -----------------------------------------------------------
usr/src/lib/libzfs/common/libzfs_util.c (line 742) <https://reviews.csiden.org/r/181/#comment458> zc_nvlist_dst is a uint64_t, so instead of NULL this should be a check for '== (uint64_t)0' or simply '== 0' same thing in the following function - Richard PALO On mars 25, 2015, 3:50 après-midi, Matthew Ahrens wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.csiden.org/r/181/ > ----------------------------------------------------------- > > (Updated mars 25, 2015, 3:50 après-midi) > > > Review request for OpenZFS Developer Mailing List, Christopher Siden and > Chris Williamson. > > > Bugs: 5745 > https://www.illumos.org/projects/illumos-gate//issues/5745 > > > Repository: illumos-gate > > > Description > ------- > > 5745 zfs set allows only one dataset property to be set at a time > Reviewed by: Christopher Siden <[email protected]\> > Reviewed by: George Wilson <[email protected]\> > Reviewed by: Matthew Ahrens <[email protected]\> > > Original author: Chris Williamson > > > Diffs > ----- > > usr/src/lib/libzfs/common/libzfs.h 9aa5ba59056455d7ecc510189546a1349d65eaa5 > usr/src/man/man1m/zfs.1m 2aa37cfdbeaef883a5a7736a42f99fc45768b4ae > usr/src/lib/libzfs/common/mapfile-vers > 65f266996e168c55d600e26895a12e0ab4cad460 > usr/src/lib/libzfs/common/libzfs_util.c > 414d74ee20660a067cf224b91677c09658a05854 > usr/src/lib/libzfs/common/libzfs_dataset.c > 05a2fdc9bec02b91a1cd48b1bf81ac581f726e28 > usr/src/cmd/zfs/zfs_main.c 5037325e9bcdb543cd16b733e4ff2137a6b37b72 > > Diff: https://reviews.csiden.org/r/181/diff/ > > > Testing > ------- > > ztest > zfs test suite > manual testing of new feature > > http://jenkins/job/zfs-precommit/1910/ > > > Thanks, > > Matthew Ahrens > >
_______________________________________________ developer mailing list [email protected] http://lists.open-zfs.org/mailman/listinfo/developer
