On 25/01/17 16:36, Wei Liu wrote: > On Wed, Jan 25, 2017 at 09:33:05AM -0700, Jan Beulich wrote: >>>>> On 25.01.17 at 16:44, <wei.l...@citrix.com> wrote: >>> --- a/tools/tests/x86_emulator/test_x86_emulator.c >>> +++ b/tools/tests/x86_emulator/test_x86_emulator.c >>> @@ -42,7 +42,7 @@ static int read( >>> struct x86_emulate_ctxt *ctxt) >>> { >>> if ( verbose ) >>> - printf("** %s(%u, %p,, %u,)\n", __func__, seg, (void *)offset, >>> bytes); >>> + printf("** %s(%u, %p, %u)\n", __func__, seg, (void *)offset, >>> bytes); >> Back when these got added, Andrew did explain why he put them >> there. >> > Eh, OK. If there is a reason for this, I will just drop this patch. It's > not essential anyway. > > It just looks rather odd to me to have so many extraneous commas. ;-)
This is my debugging shorthand, which associates values with the positional parameters. In this case it is quite trivial to read, but debugging elsewhere in the hypervisor is rather more complicated (some of the pagetable code particular takes large numbers of parameters). ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel