From: Tomas Babej Date: Sat, 15 Aug 2015 12:45:35 +0000 (+0200) Subject: TaskWarrior: Return ReadOnlyDictView of config instead of passing a copy of a mutable... X-Git-Url: https://git.madduck.net/etc/taskwarrior.git/commitdiff_plain/63bcc1eedbf3b4e6180d28c17035c964cf6a6b3f?hp=3928be287cd1cd84db996b41125435c62e03339e TaskWarrior: Return ReadOnlyDictView of config instead of passing a copy of a mutable dict --- diff --git a/tasklib/backends.py b/tasklib/backends.py index b2038be..fa5ce86 100644 --- a/tasklib/backends.py +++ b/tasklib/backends.py @@ -9,7 +9,7 @@ import six import subprocess import copy -from .task import Task, TaskQuerySet +from .task import Task, TaskQuerySet, ReadOnlyDictView from .filters import TaskWarriorFilter from .serializing import local_zone @@ -234,7 +234,7 @@ class TaskWarrior(Backend): def config(self): # First, check if memoized information is available if self._config: - return copy.deepcopy(self._config) + return self._config # If not, fetch the config using the 'show' command raw_output = self.execute_command( @@ -251,9 +251,9 @@ class TaskWarrior(Backend): config[match.group('key')] = match.group('value').strip() # Memoize the config dict - self._config = config + self._config = ReadOnlyDictView(config) - return copy.deepcopy(config) + return self._config def execute_command(self, args, config_override=None, allow_failure=True, return_all=False):