The refactoring to SetStatus and SetContentType still looks broken to me. Did you successfully push your latest stuff? If so, what are you trying to achieve with the refactoring? Maybe we can achieve it in a different way.

Note that the purpose of WriteHeader is not to allow the developer to set the status code and content type. The developer does that by assigning to c.Response.Status and c.Response.ContentType. This func is so that result implementations can specify a default status/content-type. That functionality is combined with writing the header, because that is the point at which it must be evaluated.

I would suggest leaving it as-is unless you see a simplification that meets that same need.

It's been a few days and if I recall correctly, I renamed that function because it clashes with an http.ResponseWriter one of the same name (but it's been long enough that I could be totally wrong). I'll take a look at it when I get back tonight.

Rob, somethings not right. I put those methods back as one and just renamed it so that it wouldn't clash (error below). I thought I'd pushed that change to github but it doesn't look like it. Sorry for the hassle, I'll clean it up.