Re: [PATCH] - Fix dreamcast irq.c formatting

[Kristoffer Ericson <kristoffer.ericson@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1099 bytes --]

On Wed, 19 Nov 2008 00:49:51 +0900
Paul Mundt <lethal@linux-sh.org> wrote:

> On Tue, Nov 18, 2008 at 05:28:47PM +0100, Kristoffer Ericson wrote:
> > This patch cleans the bad formatting
> > found in mach-dreamcast/irq.c file.
> > 
> Why even bother? There are more useful things you can be doing with this
> file, like fixing up the comment to deal with the fact there have been
> path changes, converting to the new IRQ model (this is one of the last
> ones left on hw_interrupt_type), etc. Once those sorts of things are out
> of the way, the rest is fairly trivial to tidy, and you can fix up the
> formatting as you go along with regards to new code.

Oki point taken. Ive got no dreamcast to fiddle with
so I'll just leave it alone. Just decided to drop a patch
when I accidently saw the mess it was in.

> 
> Whitespace changes have a high risk of causing merge conflicts and
> provide precisely zero benefit. If you don't plan to be making any
> constructive changes to this file, then simply leave it alone.


-- 
Kristoffer Ericson <kristoffer.ericson@gmail.com>

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]