<div dir="ltr"><div>Yes, you are right. There is no of use last statement.<br></div><div>Don't know how did I miss this thing.<br></div><div>I will erase the last statement and send the revision version of this again. <br>
</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Feb 20, 2014 at 1:11 AM, David Soria Parra <span dir="ltr"><<a href="mailto:davidsp@fb.com" target="_blank">davidsp@fb.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><br>
<br>
On 2/19/14, 11:27 AM, "Anurag Goel" <<a href="mailto:anurag.dsps@gmail.com">anurag.dsps@gmail.com</a>> wrote:<br>
<br>
>diff -r 76c749d59a82 -r fdb4852286de mercurial/commands.py<br>
>--- a/mercurial/commands.py    Sat Feb 15 23:58:54 2014 +0530<br>
>+++ b/mercurial/commands.py    Thu Feb 20 00:54:59 2014 +0530<br>
>@@ -5232,9 +5232,6 @@<br>
>         util.setsignalhandler()<br>
>         self.httpd = hgweb_server.create_server(self.ui, self.app)<br>
><br>
>-        if self.opts['port'] and not self.ui.verbose:<br>
>-            return<br>
>-<br>
>         if self.httpd.prefix:<br>
>             prefix = self.httpd.prefix.strip('/') + '/'<br>
>         else:<br>
>@@ -5260,6 +5257,9 @@<br>
>         write(_('listening at http://%s%s/%s (bound to %s:%d)\n') %<br>
>               (fqaddr, port, prefix, bindaddr, self.httpd.port))<br>
><br>
>+        if self.opts['port'] and not self.ui.verbose:<br>
>+            return<br>
>+<br>
<br>
</div>you are making this a no-op. Remove the last part of the patch, there is<br>
no use to adding a bogus return statement at the end.<br>
<br>
I am not sure if we want to remove the condition at all, but for<br>
consistency I would say we should always display<br>
the listening at, even when we specified a port.<br>
<br>
</blockquote></div><br></div>