On Fri, Dec 13, 2013 at 3:26 PM, Christopher <[email protected]> wrote:
> On Fri, Dec 13, 2013 at 3:55 PM, Sean Busbey <[email protected]> > wrote: > > I think the answer should be #3 and I am willing to find time to > implement > > it next week. > > That's great. In the meantime, I'm doing #4, so I can clear the noise > and focus on my own tickets. > > Could you just do #4 as a patch on your own local repo? It should be relatively easy to keep rebased given that it's just an interface. > > > Also, you should not be able to use a Connection that is attached to a > > now-closed Instance. I would also consider such use a bug. > > > [snip] > > It's not just the Connection you'd have to enforce not being re-used > after the Instance is closed... it's every object our factory-based > API can give you, all the way down. That's not a small change. > > I agree that fixing things to always expose the bug would be hard. I don't think that changes that it's a bug. -- Sean
