# HG changeset patch
# User Kostia Balytskyi <ikostia@fb.com>
# Date 1478776828 28800
# Node ID 21a75b63c10e83f3fcebf71d1f11211adb2bb3f2
# Parent  455f7856db2051db554d679249d4f4880b8ff97c
shelve: move unknown files handling to a separate function

This change has nothing to do with future obsshelve introduction,
it is done just for readability purposes.

diff -r 455f7856db20 -r 21a75b63c10e hgext/shelve.py
--- a/hgext/shelve.py	Thu Nov 10 03:07:20 2016 -0800
+++ b/hgext/shelve.py	Thu Nov 10 03:20:28 2016 -0800
@@ -335,6 +335,13 @@
                    fp=shelvedfile(repo, name, 'patch').opener('wb'),
                    opts=mdiff.diffopts(git=True))
 
+def _includeunknownfiles(repo, pats, opts, extra):
+    s = repo.status(match=scmutil.match(repo[None], pats, opts),
+                    unknown=True)
+    if s.unknown:
+        extra['shelve_unknown'] = '\0'.join(s.unknown)
+        repo[None].add(s.unknown)
+
 def _docreatecmd(ui, repo, pats, opts):
     wctx = repo[None]
     parents = wctx.parents()
@@ -364,13 +371,9 @@
                           not opts.get('addremove', False))
 
         name = getshelvename(repo, parent, opts)
-        extra={}
+        extra = {}
         if includeunknown:
-            s = repo.status(match=scmutil.match(repo[None], pats, opts),
-                            unknown=True)
-            if s.unknown:
-                extra['shelve_unknown'] = '\0'.join(s.unknown)
-                repo[None].add(s.unknown)
+            _includeunknownfiles(repo, pats, opts, extra)
 
         if _iswctxonnewbranch(repo) and not _isbareshelve(pats, opts):
             # In non-bare shelve we don't store newly created branch