> On Fri, 2006-08-18 at 18:48 +0900, Ken Hironaka wrote:
> > +const char* hypercall_name_table[64]=
> > +{
> > + "set_trap_table", //0
> > + "mmu_update", //1
> > + "set_gdt", //2
>
> ...
>
> Rather than have all those numbers in the comments, it would be better
> to use the C99 array initialization syntax:
>
> const char* hypercall_name_table[64] = {
> [__HYPERVISOR_set_trap_table] = "set_trap_table",
> [__HYPERVISOR_mmu_update] = "mmu_update",
> ...
> };
Agreed!
Also, just thought I'd be pedantic and mention we generally seek to avoid C++
style comments. We seem to have standardised on /* */ comments.
Sorry, Ken, I know it's geeky but I saw those slashes and couldn't resist!
Maybe it'll be useful for future patches.
Cheers,
Mark
> > @@ -117,10 +185,17 @@ int main(int argc, char *argv[])
> > sum += val[j];
> > printf ("T=%10u ", (unsigned int)sum);
> >
> > - if ( full || (pcd[i].nr_vals <= 4) )
> > + if ( full || (pcd[i].nr_vals <= 4) ){
> > + if( strcmp(pcd[i].name, "hypercalls") == 0 ){
> > + for( j = 0; j < pcd[i].nr_vals; j++ )
> > + printf("%s\t",
> > hypercall_name_table[j]);
> > + printf("\n");
> > + }
> > for ( j = 0; j < pcd[i].nr_vals; j++ )
> > printf(" %10u", (unsigned int)val[j]);
> >
> > + }
> > +
> > printf("\n");
> > val += pcd[i].nr_vals;
> > }
>
> Looks like the whitespace got out of hand here. Please double-check.
>
> Also, it looks like this file has had some tabs creep into it. You
> should probably send a tab-removal patch first, then this patch on top
> of that.
--
Dave: Just a question. What use is a unicyle with no seat? And no pedals!
Mark: To answer a question with a question: What use is a skateboard?
Dave: Skateboards have wheels.
Mark: My wheel has a wheel!
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
Home