# HG changeset patch # User Mads Kiilerich # Date 1360517069 -3600 # Node ID 76ff3a715cf2db991032df27e2ca0ab7044dccae # Parent 3e92772d53833323c55e2b8492facb2b0f26b0c8 hgweb: simplify internal staticfile return codes diff -r 3e92772d5383 -r 76ff3a715cf2 mercurial/hgweb/common.py --- a/mercurial/hgweb/common.py Sun Feb 10 18:24:29 2013 +0100 +++ b/mercurial/hgweb/common.py Sun Feb 10 18:24:29 2013 +0100 @@ -129,7 +129,7 @@ for part in parts: if (part in ('', os.curdir, os.pardir) or os.sep in part or os.altsep is not None and os.altsep in part): - return "" + return fpath = os.path.join(*parts) if isinstance(directory, str): directory = [directory] @@ -144,7 +144,6 @@ data = fp.read() fp.close() req.respond(HTTP_OK, ct, body=data) - return "" except TypeError: raise ErrorResponse(HTTP_SERVER_ERROR, 'illegal filename') except OSError, err: diff -r 3e92772d5383 -r 76ff3a715cf2 mercurial/hgweb/hgwebdir_mod.py --- a/mercurial/hgweb/hgwebdir_mod.py Sun Feb 10 18:24:29 2013 +0100 +++ b/mercurial/hgweb/hgwebdir_mod.py Sun Feb 10 18:24:29 2013 +0100 @@ -197,7 +197,8 @@ if isinstance(tp, str): tp = [tp] static = [os.path.join(p, 'static') for p in tp] - return (staticfile(static, fname, req),) + staticfile(static, fname, req) + return [] # top-level index elif not virtual: diff -r 3e92772d5383 -r 76ff3a715cf2 mercurial/hgweb/webcommands.py --- a/mercurial/hgweb/webcommands.py Sun Feb 10 18:24:29 2013 +0100 +++ b/mercurial/hgweb/webcommands.py Sun Feb 10 18:24:29 2013 +0100 @@ -842,7 +842,8 @@ if isinstance(tp, str): tp = [tp] static = [os.path.join(p, 'static') for p in tp] - return [staticfile(static, fname, req)] + staticfile(static, fname, req) + return [] def graph(web, req, tmpl):