outgoing: remove some early return

Since 066b8d8f75b8, the push command accept multiple destination. However `hg
outgoing` does not. On the way to fix this, we need to clean up the outgoing
code. We start with removing some early return to make the code ready to house
more changes.

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

mercurial/hg.py

@@ -1360,10 +1360,10 b' def outgoing(ui, repo, dest, opts):'
 
     limit = logcmdutil.getlimit(opts)
     o, other = _outgoing(ui, repo, dest, opts)
+    ret = 1
     try:
-        if not o:
-            cmdutil.outgoinghooks(ui, repo, other, opts, o)
-            return recurse()
+        if o:
+            ret = 0
 
         if opts.get(b'newest_first'):
             o.reverse()
@@ -1380,8 +1380,8 b' def outgoing(ui, repo, dest, opts):'
             displayer.show(repo[n])
         displayer.close()
         cmdutil.outgoinghooks(ui, repo, other, opts, o)
-        recurse()
-        return 0  # exit code is zero since we found outgoing changes
+        ret = min(ret, recurse())
+        return ret  # exit code is zero since we found outgoing changes
     finally:
         other.close()