[PATCH 1 of 9 V2] largefiles: wrap "rebase.rebase" for functions using it directly

Martin von Zweigbergk martinvonz at google.com
Wed Nov 5 12:17:36 CST 2014


On Wed Nov 05 2014 at 8:57:44 AM Martin von Zweigbergk <
martinvonz at google.com> wrote:

> On Wed Nov 05 2014 at 6:35:01 AM FUJIWARA Katsunori <foozy at lares.dti.ne.jp>
> wrote:
>
>> diff --git a/hgext/largefiles/uisetup.py b/hgext/largefiles/uisetup.py
>> --- a/hgext/largefiles/uisetup.py
>> +++ b/hgext/largefiles/uisetup.py
>> @@ -169,6 +169,8 @@
>>          if name == 'rebase':
>>              extensions.wrapcommand(getattr(module, 'cmdtable'),
>> 'rebase',
>>                  overrides.overriderebase)
>> +            extensions.wrapfunction(module, 'rebase',
>> +                                    overrides.overriderebase)
>>
>
> Do we still need to wrap the command or is it enough to wrap the function?
> Does the command call the function?
>
>

I now see how it works (the function is simply stored in the cmdtable), so
please ignore my question.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20141105/e0ec91ff/attachment.html>


More information about the Mercurial-devel mailing list