> For the "it could break stuff" argument, I agree, but it was clearly stated > by al that we can break compatibility and that the API should be clean for > 6.1.
Just because we *can* break stuff does not mean we *should* break stuff on a whim, which what happens here. > As far as I understand that, that's not because we chose a bad name at first > that we should keep it. And there's nothing bad about `#stop`. > As I explained, now the ParentedMixin is totally independent to Widget and so > there is no purpose to conceive it with Widget in mind. Widget is not independent of ParentedMixin and that's more than reason enough to consider this when you suggest breaking any code using this protocol without justification. > Finally, OF COURSE it implies destructor semantic No, destructor semantics are complex and, among other things, imply the object does not *exist* after the destructor or finalizer was called. This is *not* what this method does and it is *not* a destructor, semantically it matches C#'s Dispose() or Java's (and Python's) close(), not destructors or finalizers. [a bunch of ad hominem] > If you compare to C++ Don't compare to C++, there are no comparison basis because the languages could hardly have more different lifecycle semantics. If you want to compare to something I'd suggest comparing to other garbage-collected languages, dynamically typed or not, that would be a much better base. And none of them implements destructor semantics any more than your mis-renamed method does. > Those are fixed. Half are fixed. > Not sure I understand. Could you rephrase? Why would stopping semantics, and stoppage queries, belong in a mixin for parent/child relationship instead of being independent from and above it? -- https://code.launchpad.net/~openerp-dev/openerp-web/trunk-core-extraction-1-niv/+merge/94021 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openerp-web/trunk-core-extraction-1-niv. _______________________________________________ Mailing list: https://launchpad.net/~openerp-dev-gtk Post to : [email protected] Unsubscribe : https://launchpad.net/~openerp-dev-gtk More help : https://help.launchpad.net/ListHelp
