On Tue, Sep 6, 2011 at 2:05 PM, Brion Vibber <br...@pobox.com> wrote: > Generally speaking, we should only throw warnings or remove old interfaces > that are actively broken (do not work correctly) or can no longer be sanely > maintained -- removing a deprecated interface is a fairly extreme step and > should never be done just to make things look cleaner. > > There may be little or even *negative* benefit to going around and changing > all the calling code to use the new interface. I've seen *lots* of > regressions in commits that swap something to a new interface without taking > into account how the interface actually changed, and they're harder to track > down because the changes are often buried in generic code clean-up.
+1000. Rob _______________________________________________ Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l