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
