Message195811
Backward compatibility is a concern. However, having the flags test true when present, which they don't currently, should be considered a bug fix, I think.
So, I added a some tests to confirm the current behavior, and wrote a new patch that *just* fixes the 'test true if present' bug, leaving the other behaviors unchanged. Reviews appreciated. And if anyone can think of tests I missed, please let me know; I'm a little nervous about the regex change (I hate dealing with regexes :)
This issue was originally about other bugs as well, but I'm changing the title to reflect just this fix. If there are still other remaining bugs, please open a new issues for them.
Thanks for the patch, Julien. Although I didn't use your code, it did inform my work. |
|
Date |
User |
Action |
Args |
2013-08-21 18:08:53 | r.david.murray | set | recipients:
+ r.david.murray, ezio.melotti, eric.araujo, flox, jdennis, julien.phalip |
2013-08-21 18:08:53 | r.david.murray | set | messageid: <1377108533.46.0.321681030597.issue16611@psf.upfronthosting.co.za> |
2013-08-21 18:08:53 | r.david.murray | link | issue16611 messages |
2013-08-21 18:08:52 | r.david.murray | create | |
|