[PATCH v2] python: deprecated add_message calls index_file correctly and returns result
Gaute Hope
eg at gaute.vetsj.com
Wed Aug 30 01:16:33 PDT 2017
The deprecated Database.add_message now calls the new index_file with
correct number of arguments (without an extra `self`), and returns the
tuple from index_file - as it used to do before.
This change also adds a DeprecationWarning to the function.
---
bindings/python/notmuch/database.py | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/bindings/python/notmuch/database.py b/bindings/python/notmuch/database.py
index a2c025eb..1279804a 100644
--- a/bindings/python/notmuch/database.py
+++ b/bindings/python/notmuch/database.py
@@ -19,6 +19,7 @@ Copyright 2010 Sebastian Spaeth <Sebastian at SSpaeth.de>
import os
import codecs
+import warnings
from ctypes import c_char_p, c_void_p, c_uint, byref, POINTER
from .compat import SafeConfigParser
from .globals import (
@@ -471,7 +472,10 @@ class Database(object):
def add_message(self, filename, sync_maildir_flags=False):
"""Deprecated alias for :meth:`index_file`
"""
- self.index_file(self, filename, sync_maildir_flags=sync_maildir_flags)
+ warnings.warn(
+ "This function is deprecated and will be removed in the future, use index_file.", DeprecationWarning)
+
+ return self.index_file(filename, sync_maildir_flags=sync_maildir_flags)
_remove_message = nmlib.notmuch_database_remove_message
_remove_message.argtypes = [NotmuchDatabaseP, c_char_p]
--
2.14.1
More information about the notmuch
mailing list