diff extensions/kwarchive.py @ 136:f470702886b4

Renamed some internal functions to provide a more descriptive name
author Franz Glasner <hg@dom66.de>
date Mon, 20 Aug 2018 09:18:23 +0200
parents aa553e987939
children 5e502e9dd5e5
line wrap: on
line diff
--- a/extensions/kwarchive.py	Mon Aug 20 09:14:37 2018 +0200
+++ b/extensions/kwarchive.py	Mon Aug 20 09:18:23 2018 +0200
@@ -221,8 +221,8 @@
                 ac,
                 archival.tidyprefix(dest, kind, prefix),
                 hgpath=opts.get("path"),
-                path_filter=get_checked_path_filter(opts),
-                user_filter=get_checked_user_filter(opts),
+                path_filter=get_checked_path_filter_option(opts),
+                user_filter=get_checked_user_filter_option(opts),
                 kwconfig=opts.get("kwconfig")))
 
     archival.archive(repo, dest, node, kind, not opts.get('no_decode'),
@@ -301,8 +301,8 @@
     keywords = make_node_keywords(
         ui, repo, ctx, prefix,
         hgpath=opts.get("path"),
-        path_filter=get_checked_path_filter(opts),
-        user_filter=get_checked_user_filter(opts))
+        path_filter=get_checked_path_filter_option(opts),
+        user_filter=get_checked_user_filter_option(opts))
     # make file-dependent keywords for an example file
     file_keywords = make_file_keywords(
         keywords,
@@ -576,7 +576,7 @@
     return hostname[:idx]
 
 
-def get_checked_user_filter(opts):
+def get_checked_user_filter_option(opts):
     uf = opts.get("user_filter")
     # "none" is retained for compatibility reasons and now an alias for "full"
     if uf not in ("person", "user", "email", "full", "none"):
@@ -585,7 +585,7 @@
     return uf
 
 
-def get_checked_path_filter(opts):
+def get_checked_path_filter_option(opts):
     pf = opts.get("path_filter")
     if pf not in ("full", "nopwd", "short", "last"):
         raise error.Abort(