Comment 17 for bug 358009

Revision history for this message
Bryce Harrington (bryce) wrote :

Boy... on the one hand I hate to be putting in non-trivial changes this close to freeze, but on the other hand you've clearly done some good work chasing this down and it would be a shame to leave it out.

Normally, I'd want to see this incorporated into the upstream tree, however at least you've gotten the change reviewed by whot, and it sounds like he didn't spot obvious flaws, just that he has larger plans in motion. I certainly don't mind papering over issues - anything to make crashes go away!

I'd like to have seen it more widely tested as well, but your analysis and testing sounds like it was done carefully. I'll incorporate it, provisionally that it is not implicated with any regressions. We can always revert the patch if we find problems with it.