Comment 4 for bug 363873

Revision history for this message
Olivier Tilloy (osomon) wrote :

Hi Andrea,
The patch seems to make sense from what I can read, and I can't recall why this test is here in the first place.
It doesn't break the unit tests, but on the other hand unit tests don't really check thoroughly this piece of code.
I'm not against merging the patch, but I'm concerned about potential regressions we cannot spot right now.
I'm currently focusing my efforts on putting the finishing touch to the 0.2 branch to release it as soon as possible, in the hope that I can drop maintenance of the 0.1 branch which proved to be buggy in various places and difficult to maintain.
If you are willing to do some more testing, I certainly wouldn't mind merging the patch, and we can then propose it for inclusion in the debian/ubuntu packages.