From mboxrd@z Thu Jan 1 00:00:00 1970 From: john at keeping.me.uk (John Keeping) Date: Sun, 7 Apr 2013 12:25:43 +0100 Subject: [PATCH 02/19] Remove redundant calls to fmt("%s", ...) In-Reply-To: References: <55887ea74c707fbc83f9a6e2527b18c9566b7198.1365326321.git.john@keeping.me.uk> Message-ID: <20130407112543.GV2222@serenity.lan> On Sun, Apr 07, 2013 at 01:05:40PM +0200, Jason A. Donenfeld wrote: > On Sun, Apr 7, 2013 at 11:29 AM, John Keeping wrote: > > - ctx.page.filename = fmt("%s", path); > > + ctx.page.filename = "%s"; > > These aren't exactly equivalent. Is there something I'm missing here > (I'm going through this patch set linearly.)? That I was on autopilot at this stage and didn't check this thoroughly enough :-( Clearly it should be: + ctx.page.filename = path;