On Wed, 16 Sep 2009 11:51:17 +0100 Alan Maguire <Alan.Maguire at Sun.COM> wrote:
> webrev at: > > http://zhadum.east.sun.com/export/ws/amaguire/nwam1-fixes/webrev/ These look good. One nit: enm.c:138,141,144 you can rearrange this code to use an intermediate for the return value of nwam_value_get_uint64 and avoid having the free in two places. mph > > thanks! > > Alan > _______________________________________________ > nwam-dev mailing list > nwam-dev at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/nwam-dev
