Move finding/checking the log limit to cmdutil

mercurial/cmdutil.py

@@ -89,6 +89,19 b' def logmessage(opts):'
                              (logfile, inst.strerror))
     return message
 
+def loglimit(opts):
+    """get the log limit according to option -l/--limit"""
+    limit = opts.get('limit')
+    if limit:
+        try:
+            limit = int(limit)
+        except ValueError:
+            raise util.Abort(_('limit must be a positive integer'))
+        if limit <= 0: raise util.Abort(_('limit must be positive'))
+    else:
+        limit = sys.maxint
+    return limit
+
 def setremoteconfig(ui, opts):
     "copy remote options to ui tree"
     if opts.get('ssh'):

mercurial/commands.py

@@ -1699,14 +1699,7 b' def log(ui, repo, *pats, **opts):'
     get = util.cachefunc(lambda r: repo.changectx(r).changeset())
     changeiter, matchfn = cmdutil.walkchangerevs(ui, repo, pats, get, opts)
 
-    if opts['limit']:
-        try:
-            limit = int(opts['limit'])
-        except ValueError:
-            raise util.Abort(_('limit must be a positive integer'))
-        if limit <= 0: raise util.Abort(_('limit must be positive'))
-    else:
-        limit = sys.maxint
+    limit = cmdutil.loglimit(opts)
     count = 0
 
     if opts['copies'] and opts['rev']: