Michael Hunter wrote: > 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. > > done (and webrev updated). Thanks for the review!
Alan
