Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > comp.lang.python > #105469 > unrolled thread

Re: monkey patching __code__

Back to article view | Back to comp.lang.python

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: monkey patching __code__ "Sven R. Kunze" <srkunze@mail.de> - 2016-03-22 13:01 +0100

#105469 — Re: monkey patching __code__

On 21.03.2016 21:42, Matt Wheeler wrote:
> On 20 March 2016 at 16:46, Sven R. Kunze <srkunze@mail.de> wrote:
>> On 19.03.2016 00:58, Matt Wheeler wrote:
>>> I know you have a working solution now with updating the code &
>>> defaults of the function, but what about just injecting your function
>>> into the modules that had already imported it after the
>>> monkeypatching?
>>>
>>> Seems perhaps cleaner, unless you'd end up having to do it to lots of
>>> modules...
>> Why do you consider it cleaner?
> I think it would be more explicit and understandable for someone
> reading your code.
>
> I suppose it's quite subjective :)

As far as I can see, the code replacement approach solves the problem 
once and for all. Thus is far more stable.

Manually finding out every single module that might or might not have 
imported "reverse" before we could monkeypatch it might result in a 
maintenance nightmare (just think about a Django upgrade).


It reminds me of list replacement:

mylist = newlist
mylist[:] = newlist

The latter keeps the reference stable whereas the former does not. Same 
with monkeypatching.


Best,
Sven

[toc] | [standalone]

Back to top | Article view | comp.lang.python

csiph-web