On Apr 2, 2013, at 8:49 PM, Ben Pfaff <blp at nicira.com> wrote: > - if (want_report) { > + if (want_report && s.length > 0) { Isn't our style normally to do it like this? if (want_report && s.length) { Otherwise, looks good. --Justin