[PATCH] python: Fix database.add_message
Gaute Hope
eg at gaute.vetsj.com
Tue Sep 12 00:40:08 PDT 2017
David Bremner writes on september 12, 2017 3:56:
> Dylan Baker <dylan at pnwbakers.com> writes:
>
>> b10ce6bc23002d48916b1b2f375480e7540e3164 introduced a new index_file
>> method as a replacement for add_message. It helpfully made add_message
>> an alias for index_file, unfortunately it got the signature wrong by
>> explicitly passing self as an argument.
>>
>> cc: Daniel Kahn Gillmor <dkg at fifthhorseman.net>
>
> Thanks. We have another version that I was about to commit
>
> https://www.mail-archive.com/notmuch@notmuchmail.org/msg45038.html
>
> The only difference is that Gaute (more or less by request) added a
> deprecation warning. I don't really know how warnings in python work,
> does that sound sensible?
Yup, also notice the missing `return`.
Regards, Gaute
More information about the notmuch
mailing list