> I'm committed to seeing this patch to either oblivion or production,
> and globally altering whitespace is going to blow my time for *no net
> benefit to anyone*.

Cleanups like these _do_ have benefit.


>       If you really are that wholly uninterested in a
> complete scheduler,

You're overreacting more than a little bit.  We want that, and also
want good clean code.  It's not as if this patch was submitted to
annoy you (or anyone).


>       I'll just give up now and we'll ship the kernel
> patch.  I don't mind jumping through hoops to work nicely with others
> [really; I expect the review process for these changes to suck, but
> that's the only reasonable path], but if my time is really worth
> nothing whatsoever,

Overreacting again.  You know, everybody _else_ has to cope with
patch conflicts all the time ... and it's not like I haven't held
this cleanup patch back for a while now (ditto one for OHCI).


>               that I'm expected to refactor because tabs v.
> spaces is more important that working audio devices, I have to think
> hard about just how much I care.  My USB devices work fine now, and I
> can direct customers to an internal patchset if they need it.

Remember that you still haven't provided a _factored_ patch, as in
"incrementally reviewable changes".  One Big Unreviewable Patch is
what you've talked about so far.

I'm sure you'll find that once your patch is factored into a series
of incremental improvements, whitespace conflicts for any one of
those increments will be minor noise.  And for that matter, it'd
be easy enough for you to "patch -R" conflicty bits of this patch
in your copy of the tree, then just go fix the whitespace up again
after applying your patch.  There are LOTS of ways to cope with
things like this.

- Dave


-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to