On Nov. 08, 2007, 5:13 +0200, FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx> wrote: > On Tue, 06 Nov 2007 20:16:19 +0200 > Boaz Harrosh <bharrosh@xxxxxxxxxxx> wrote: > >> - If we export scsi_init_io()/scsi_release_buffers() instead of >> scsi_{alloc,free}_sgtable() from scsi_lib than tgt code is >> much more insulated from scsi_lib changes. As a bonus it will >> also gain bidi capability when it comes. >> >> Signed-off-by: Boaz Harrosh <bharrosh@xxxxxxxxxxx> > > Looks good for me except for this: > > ./scripts/checkpatch.pl ~/Mail/kernel/scsi/28814 > ERROR: use tabs not spaces > #101: FILE: drivers/scsi/scsi_lib.c:741: > + gfp_t gfp_mask)$ Come on Tomo, tabs should be used for nesting, not for decoration. This way no matter what's your tab expansion setup is the code will look correct and will make sense. The number of space characters in this case is equal to the number of characters in the line above and with fixed-width fonts the line will be indented just as you wanted. The bottom line is that you should indent with as many tabs as your nesting level, where all statements will begin, and from there on use space characters. For example: { if (very_long_expression && it_needs_to_be_broken_into_several_lines) return a_very_long_result + the_remainder_of_it_that_spilled_off + to_the_next_lines; return printk("just my %d cents\n", 2); } Benny > > total: 1 errors, 0 warnings, 144 lines checked - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html