> > > > @@ -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