[Twisted-Python] svn-reorg blocking
Jonathan Lange
jml at mumak.net
Tue Oct 25 22:10:04 MDT 2005
On 26/10/05, glyph at divmod.com <glyph at divmod.com> wrote:
> >On 10/26/05, James Y Knight wrote <foom at fuhm.net> wrote:
>
> >The way I see it is that you put a horrible hack in trial to work
> >around a problem that wouldn't happen if the code actually followed
> >Twisted's coding convention.
>
> In James' defense, Twisted's coding standard actually says what serialport is doing is OK. The portion of the standard that says this is my work, and I think the coding standard should be changed, because this is an awful hack, but one thing that can't be said about it is that it isn't officially endorsed :).
>
> I tend to agree with your general sentiment but I don't think that this issue is important enough to hold up the re-org. James has implemented a solution. Let's just use it.
I strongly and vigorously oppose merging this branch until there are
unit tests for James' modifications to Trial. Instead I recommend
moving serialport up a level (as you suggested earlier). That appears
to be the simplest thing that could possibly work.
cheers,
jml
More information about the Twisted-Python
mailing list