Robert Bradshaw, 06.12.2010 19:37: > On Sun, Dec 5, 2010 at 7:55 AM, Stefan Behnel wrote: >> mark florisson, 05.12.2010 15:11: >>> On 5 December 2010 14:15, Stefan Behnel wrote: >>>> I haven't tested it yet, mainly because I don't have the latest gdb version >>>> installed. When I tried building lxml with that branch, I ran into a couple >>>> of compiler crashes that Mark resolved, but I can't tell if there aren't >>>> any left. I also didn't review the code in any way. >>>> >>>> AFAIC, I'm not sure I want this right in 0.14, but that's more because I >>>> can't really comment on it. I certainly agree that it needs more testing. >>> [...] >>> But I agree that it should be tested more widely before merging it. The >>> documentation (currently only on hudson but I presume it will make it >>> to docs.cython.org with the new release) already contains a link to my >>> git branch so users that are interested could still use it. >> >> Then I would suggest we keep the docs as they are now to make your gdb >> support visible, but with a big note that this hasn't been merged yet and >> keep your branch alive for a bit longer after the release. I wouldn't mind >> merging it for 0.14.1 when it seems mature enough, and I hope that release >> isn't too far off. 0.14 is a release with many, many new features, so it's >> quite likely that people find bugs quickly once it's out in the wild. > > OTOH, when the flag is off, the changes made are quite noninvasive, so > I would be fine merging this as an experimental feature to get more > widespread testing and for what utility it already has, even if it's > not fully polished yet (not that I have any bugs to point at...).
Ok, whatever. If it doesn't hurt by default, just merge it in. It's a nice feature that we can just keep growing as people start using it. Stefan _______________________________________________ Cython-dev mailing list [email protected] http://codespeak.net/mailman/listinfo/cython-dev
