[Twisted-Python] Improved StandardIO support in Windows
glyph at twistedmatrix.com
Fri Apr 19 16:35:41 EDT 2013
On Apr 19, 2013, at 12:58 PM, exarkun at twistedmatrix.com wrote:
> On 06:55 pm, johnnypops at gmail.com wrote:
>> On Fri, Apr 19, 2013 at 02:21:48PM -0400, Itamar Turner-Trauring wrote:
>>> The main issue is that the patch is too big? The easiest way to solve
>>> is to split it up into multiple tickets, each addressing a subsection
>>> the problem, with a corresponding small patch. It's fine to say "and
>>> patch requires ticket #1234 to be fixed."
>> OK, thanks, I'll do that.
> To add to what Itamar wrote a bit, an important part of this strategy is
> that the individual pieces the task gets split into need to each make
> some kind of sense on their own. It's fine for there to be dependencies
> (but the dependencies should only go in one direction), but a ticket the
> summary of which is "apply this patch to _win32stdio.py that I wrote"
> won't work very well. Instead, be sure that the tickets are things like
> "implement feature foo for Win32EventsReactor".
And to refine this even further:
It's OK for the patches to be smaller than "implement feature foo", as long as they make sense on their own.
For example if we already implement feature foo, but you want to re-factor it so that it's easy to implement feature bar in terms of that same code, you could have a ticket just for doing the inner refactoring, that does not expose any new public classes or features, as long as that refactoring has a good explanation and a good impact on the internal API.
Based on the last patch I see on this ticket, it looks like that's what you're trying to do with a lot of the 'Channel' objects; these need better documentation and test coverage, but it looks like some of that work could be off on its own.
More information about the Twisted-Python