webcommands: stop using ersatz if-else ternary operator for rename variable

6ddc86eedc3b didn't remove it, let's do it now.

Placing the added lines into the already existing "if fctx is not None" block
also makes webcommands.comparison() look a bit more like
webcommands.filediff(), which eases possible future refactoring. And fctx is
not None only when path in ctx, so logically it's equivalent.

mercurial/hgweb/webcommands.py

@@ -838,7 +838,6 b' def comparison(web, req, tmpl):'
     if 'file' not in req.form:
         raise ErrorResponse(HTTP_NOT_FOUND, 'file not given')
     path = webutil.cleanpath(web.repo, req.form['file'][0])
-    rename = path in ctx and webutil.renamelink(ctx[path]) or []
 
     parsecontext = lambda v: v == 'full' and -1 or int(v)
     if 'context' in req.form:
@@ -875,8 +874,10 b' def comparison(web, req, tmpl):'
 
     comparison = webutil.compare(tmpl, context, leftlines, rightlines)
     if fctx is not None:
+        rename = webutil.renamelink(fctx)
         ctx = fctx
     else:
+        rename = []
         ctx = ctx
     return tmpl('filecomparison',
                 file=path,