[PATCH 4/4] repolist: add a git link on front page
Sebastian Andrzej Siewior
sebastian at breakpoint.cc
Sat Feb 1 15:53:25 CET 2014
On 19.01.14, Jason A. Donenfeld wrote:
> On Sat, Jan 18, 2014 at 9:25 PM, Sebastian Andrzej Siewior
> <sebastian at breakpoint.cc> 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
More information about the CGit
mailing list