From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Date: Sun, 27 Sep 2009 09:25:11 -0500 Subject: [U-Boot] [PATCH] OMAP3: Clean up whitespace in mux configs In-Reply-To: <20090927143342.GA7850@lixom.net> References: <20090922225314.GA16136@lixom.net> <4ABF2376.5080408@googlemail.com> <20090927143342.GA7850@lixom.net> Message-ID: <4ABF75C7.3090706@windriver.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Olof Johansson wrote: > On Sun, Sep 27, 2009 at 10:33:58AM +0200, 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 ;) > > Haha, great. > >> Last time we had: >> >> ... >> [So Jan 25 2009] [18:50:28] wdenk: Regarding your code comments >> in http://lists.denx.de/pipermail/u-boot/2009-January/046090.html: >> [So Jan 25 2009] [18:51:34] wdenk: Regarding MUX indent: >> normally yes, but ... >> [So Jan 25 2009] [18:51:59] ...you have too long lines, then. I >> know >> [So Jan 25 2009] [18:52:49] 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] 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. > > Heh. > > Personally I don't mind the occasional breakage of the 80 character > rule, especially for header files like these that don't affect code > flow readability. > > But I don't care either way, I just reformatted it on request. If the > decision goes the other way I can repost the other patch that went on > top of this so it still applies. > I am ok with it as-is. If it needs changing, a follow-on is preferred. Tom > > -Olof > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot