On Wed, May 11, 2016 at 05:50:09PM +0000, Juuso Lapinlampi wrote: > --- > This may be a little controversial, but right now is there a reason not > to do this?
No, I think this makes sense (although again we need your signed-off-by). Reviewed-by: John Keeping <[email protected]> > ui-shared.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/ui-shared.c b/ui-shared.c > index 9a38aa9..d9541f0 100644 > --- a/ui-shared.c > +++ b/ui-shared.c > @@ -792,13 +792,11 @@ void cgit_print_error_page(int code, const char *msg, > const char *fmt, ...) > ctx.page.expires = ctx.cfg.cache_dynamic_ttl; > ctx.page.status = code; > ctx.page.statusmsg = msg; > - cgit_print_http_headers(); > - cgit_print_docstart(); > - cgit_print_pageheader(); > + cgit_print_layout_start() > va_start(ap, fmt); > cgit_vprint_error(fmt, ap); > va_end(ap); > - cgit_print_docend(); > + cgit_print_layout_end(); > } > > void cgit_print_layout_start(void) > -- > 2.8.1 > > _______________________________________________ > CGit mailing list > [email protected] > http://lists.zx2c4.com/mailman/listinfo/cgit _______________________________________________ CGit mailing list [email protected] http://lists.zx2c4.com/mailman/listinfo/cgit
