Hi Wolfram, On Wed, 16 Jul 2014 19:23:09 +0200, Wolfram Sang wrote: > On Mon, Jun 23, 2014 at 10:36:58AM +0200, Jean Delvare wrote: > > A long name broke the alignment, shift the columns a bit to fix it and > > make the table look nice again. While we're here, switch to the > > standard comment style to make checkpatch happy. > > > > Signed-off-by: Jean Delvare <jdelvare@xxxxxxx> > > Cc: James Ralston <james.d.ralston@xxxxxxxxx> > > Cc: Wolfram Sang <wsa@xxxxxxxxxxxxx> > > Hmmm, what about giving the names even more space so we have some buffer for the > future? And what about replacing spaces with tabs? I considered it, but could not make up my mind. The table already spans to column 74. This does not leave that much room for growing (table) columns. I am not sure if we would rather use the remaining space to have some margin for the name column, as you suggest, or keep it as spare space for an extra column if future chipsets ever implement a new feature. Currently, columns are aligned on 3, 27, 36, 44, 52, 60 and 68. None of these values are multiple of 8, so the use of tabs would not help much, unless we shift everything first. One possibility is to switch to tabs and shift all columns (but the first) by 4 to the right. This rules out the addition of a column in the future, but otherwise fulfills your desire. To be honest, I don't really care. All I wanted is that columns are properly aligned. If the plan above makes you happier then fine with me, I'll do that. -- Jean Delvare SUSE L3 Support -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
