Dirk Behme wrote: > Olof Johansson wrote: >> Switch from space-based indentation to tab-based in mux configs, as pointed >> out by WD at: > > You have only two options here: Violate space/tab rule or 80 char > rule. We had this discussion with Wolfgang already in the past ;) > > Last time we had: > > ... > [So Jan 25 2009] [18:50:28] <dirk2> wdenk: Regarding your code > comments in > http://lists.denx.de/pipermail/u-boot/2009-January/046090.html: > [So Jan 25 2009] [18:51:34] <dirk2> wdenk: Regarding MUX indent: > normally yes, but ... > [So Jan 25 2009] [18:51:59] <wdenk_> ...you have too long lines, then. > I know > [So Jan 25 2009] [18:52:49] <dirk2> wdenk: ... we tried several > formatting options, and jkridner converted one MUX file with beginning > TABs. The result wasn't readable better > [So Jan 25 2009] [18:53:34] <wdenk_> I'll not fight about this one. > ... > > So last time we stayed with space instead of tab and have 80 char fine. > > If we want to switch now to break 80 chars (and this patch does so) > but have no spaces any more, I'm fine with it. Just want to give some > history. > > Best regards >
From reviewing the patch, it looks like the mux args line up better. So it will make scanning them easier on the developer. Tom > Dirk > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
