Thanks for the review, I pushed this.

Ethan

On Fri, Aug 12, 2011 at 15:26, Ben Pfaff <b...@nicira.com> wrote:
> On Fri, Aug 12, 2011 at 03:22:52PM -0700, Ethan Jackson wrote:
>> I found this patch useful in tracking down a bug recently.
>
> Good idea.
>
> Regs have type uint32_t so the right format specifier is "%"PRIx32 (not
> %x).
>
> I think you probably don't want to make it print trailing white space in
> case we ever want to put the full output into a test case.
>
> Otherwise looks good, thank you.
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to