This adds some documentation to all the defvars in notmuch-pick and to the notmuch-pick function itself (as suggested by David in id:87li5rtvbh.fsf@zancas.localnet. This series replaces id:1372627117-31188-1-git-send-email-markwalters1009@gmail.com but does still need to be applied on top of the main series there (because it documents the pick function as it is after that change). When documenting the defvars I found some trivial bugs: the most serious being the wrong variable name being made buffer-local. This is less serious than it could be as this variable was just used to try and avoid closing the message pane if the user had loaded a different buffer into it. The other "bugs" were defvars for variables which are no longer used. In one case a moderate amount of effort was made passing this unused variable around so the patch to remove it is not tiny and thus its removal is delayed until a followup series. Finally, when looking at which functions did not have docstrings (some still need to be added after this series) I found an unused historical function so remove that too. Best wishes Mark Mark Walters (5): contrib: pick: add a docstring for the main notmuch-pick function contrib: pick: add docstrings to all defvars contrib: pick: bugfix: make the right variable buffer-local contrib: pick: remove unused variable contrib: pick: remove unused function contrib/notmuch-pick/notmuch-pick.el | 71 ++++++++++++++++++++------------- 1 files changed, 43 insertions(+), 28 deletions(-) -- 1.7.9.1