>
>
> > @@ -147,11 +141,10 @@ int kdbnearsym(unsigned long addr, kdb_symtab_t
> *symtab)
> >
> >         if (symtab->mod_name == NULL)
> >                 symtab->mod_name = "kernel";
> > -       if (KDB_DEBUG(AR))
> > -               kdb_printf("kdbnearsym: returns %d
> symtab->sym_start=0x%lx, "
> > -                  "symtab->mod_name=%px, symtab->sym_name=%px (%s)\n",
> ret,
> > -                  symtab->sym_start, symtab->mod_name, symtab->sym_name,
> > -                  symtab->sym_name);
> > +       kdb_dbg_printf(AR, "returns %d symtab->sym_start=0x%lx, "
> > +               "symtab->mod_name=%px, symtab->sym_name=%px (%s)\n", ret,
> > +               symtab->sym_start, symtab->mod_name, symtab->sym_name,
> > +               symtab->sym_name);
>
> This indention doesn't line up, but it didn't before.  I guess I'd let
> Daniel say if he likes this or would prefer different wrapping /
> indentation.
>

Thanks for your patience. You told me that  the alignment for continued
arguments is to
match up with the first argument, so I was confused that the first line and
the second line
here don't follow the rule.  There are many  cases like this in  this file.

_______________________________________________
Kgdb-bugreport mailing list
Kgdb-bugreport@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kgdb-bugreport

Reply via email to