dirstate: group return logic and clarify each function in flagfunc

As suggested by spectral during review of my fix of the missing return, this
makes it more obvious that all cases are covered and improves readability of
the functions with new names and a small docstring.

Differential Revision: https://phab.mercurial-scm.org/D11715

mercurial/dirstate.py

@@ -242,9 +242,11 b' class dirstate(object):'
         return self._rootdir + f
 
     def flagfunc(self, buildfallback):
-        if self._checklink and self._checkexec:
+        if not (self._checklink and self._checkexec):
+            fallback = buildfallback()
 
-            def f(x):
+        def check_both(x):
+            """This platform supports symlinks and exec permissions"""
                 try:
                     st = os.lstat(self._join(x))
                     if util.statislink(st):
@@ -255,12 +257,8 b' class dirstate(object):'
                     pass
                 return b''
 
-            return f
-
-        fallback = buildfallback()
-        if self._checklink:
-
-            def f(x):
+        def check_link(x):
+            """This platform only supports symlinks"""
                 if os.path.islink(self._join(x)):
                     return b'l'
                 entry = self.get_entry(x)
@@ -271,10 +269,8 b' class dirstate(object):'
                     return b'x'
                 return b''
 
-            return f
-        if self._checkexec:
-
-            def f(x):
+        def check_exec(x):
+            """This platform only supports exec permissions"""
                 if b'l' in fallback(x):
                     return b'l'
                 entry = self.get_entry(x)
@@ -285,10 +281,10 b' class dirstate(object):'
                     return b'x'
                 return b''
 
-            return f
-        else:
-
-            def f(x):
+        def check_fallback(x):
+            """This platform supports neither symlinks nor exec permissions, so
+            check the fallback in the dirstate if it exists, otherwise figure it
+            out the more expensive way from the parents."""
                 entry = self.get_entry(x)
                 if entry.has_fallback_symlink:
                     if entry.fallback_symlink:
@@ -300,7 +296,14 b' class dirstate(object):'
                         return b''
                 return fallback(x)
 
-            return f
+        if self._checklink and self._checkexec:
+            return check_both
+        elif self._checklink:
+            return check_link
+        elif self._checkexec:
+            return check_exec
+        else:
+            return check_fallback
 
     @propertycache
     def _cwd(self):