Message229888
More likely the lack of discussion is just that Serhiy is busy :)
Breaking code is to be avoided if possible. Can you give an example of the "cosmetic" change?
I haven't fully reviewed the patch, but a more meaningful name than 'condition' might make the code more readable. Perhaps 'handling_negative_limit'? |
|
Date |
User |
Action |
Args |
2014-10-23 18:40:33 | r.david.murray | set | recipients:
+ r.david.murray, serhiy.storchaka, vaultah |
2014-10-23 18:40:33 | r.david.murray | set | messageid: <1414089633.87.0.775893176901.issue22619@psf.upfronthosting.co.za> |
2014-10-23 18:40:33 | r.david.murray | link | issue22619 messages |
2014-10-23 18:40:33 | r.david.murray | create | |
|