From mboxrd@z Thu Jan 1 00:00:00 1970 From: sebastian at breakpoint.cc (Sebastian Andrzej Siewior) Date: Sat, 1 Feb 2014 15:53:25 +0100 Subject: [PATCH 4/4] repolist: add a git link on front page In-Reply-To: References: <1390076700-16626-1-git-send-email-sebastian@breakpoint.cc> <1390076700-16626-5-git-send-email-sebastian@breakpoint.cc> Message-ID: <20140201145325.GA23572@breakpoint.cc> On 19.01.14, Jason A. Donenfeld wrote: > On Sat, Jan 18, 2014 at 9:25 PM, Sebastian Andrzej Siewior > wrote: > > cgit_tree_link("tree", NULL, "button", NULL, NULL, NULL); > > + cgit_git_link(); > > @@ -284,6 +284,41 @@ void cgit_tree_link(const char *name, const char *title, const char *class, > > +void cgit_git_link(void) > > I like this, but it's probably best to refactor it to take the same > arguments as all the other cgit_*_link functions. A little hint please. I could change this to |void cgit_git_link(const char *name, const char *class) and update the caller. Is it this what you mean? If you want me to reuse site_link() (and its arguemnts) I would ne to replace site_url() somehow because it creates script based arguements. Sebastian