diff --git a/mercurial/dirstate.py b/mercurial/dirstate.py
--- a/mercurial/dirstate.py
+++ b/mercurial/dirstate.py
@@ -416,12 +416,12 @@ class dirstate(object):
                 return True
         return False
 
-    def walk(self, files=None, match=util.always, badmatch=None):
+    def walk(self, files, match, badmatch):
         # filter out the stat
         for src, f, st in self.statwalk(files, match, badmatch=badmatch):
             yield src, f
 
-    def statwalk(self, files=None, match=util.always, unknown=True,
+    def statwalk(self, files, match, unknown=True,
                  ignored=False, badmatch=None, directories=False):
         '''
         walk recursively through the directory tree, finding all files
diff --git a/mercurial/localrepo.py b/mercurial/localrepo.py
--- a/mercurial/localrepo.py
+++ b/mercurial/localrepo.py
@@ -931,7 +931,7 @@ class localrepository(repo.repository):
                 self.dirstate.invalidate()
             del tr, lock, wlock
 
-    def walk(self, node=None, files=[], match=util.always, badmatch=None):
+    def walk(self, node, files, match, badmatch):
         '''
         walk recursively through the directory tree or a given
         changeset, finding all files matched by the match
@@ -970,7 +970,7 @@ class localrepository(repo.repository):
                     self.ui.warn(_('%s: No such file in rev %s\n')
                                  % (self.pathto(fn), short(node)))
         else:
-            for src, fn in self.dirstate.walk(files, match, badmatch=badmatch):
+            for src, fn in self.dirstate.walk(files, match, badmatch):
                 yield src, fn
 
     def status(self, node1=None, node2=None, files=[], match=util.always,