List for cgit developers and users
 help / color / mirror / Atom feed
From: john at keeping.me.uk (John Keeping)
Subject: [RFC/PATCH 0/6] Preparation for more filter types
Date: Sun, 12 Jan 2014 17:13:47 +0000	[thread overview]
Message-ID: <cover.1389546691.git.john@keeping.me.uk> (raw)

This is the preliminary refactoring for supporting more types of filter
(for example Lua scripts or persistent filters).  The final patch adds a
table where more implementations can be added.

The first three (maybe four) patches are sensible cleanups even if we
don't want to take the whole plan any further.

John Keeping (6):
  html: remove redundant htmlfd variable
  ui-snapshot: set unused cgit_filter fields to zero
  filter: pass extra arguments via cgit_open_filter
  filter: add fprintf_filter function
  filter: add interface layer
  filter: introduce "filter type" prefix

 cgit.c        |   6 +--
 cgit.h        |  12 +++++-
 cgitrc.5.txt  |   9 +++++
 filter.c      | 119 ++++++++++++++++++++++++++++++++++++++++++++++++----------
 html.c        |   4 +-
 ui-repolist.c |  10 ++---
 ui-snapshot.c |   9 ++---
 ui-summary.c  |  13 +++----
 ui-tree.c     |   7 ++--
 9 files changed, 140 insertions(+), 49 deletions(-)

-- 
1.8.5.226.g0d60d77



             reply	other threads:[~2014-01-12 17:13 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-12 17:13 john [this message]
2014-01-12 17:13 ` [PATCH 1/6] html: remove redundant htmlfd variable john
2014-01-12 19:11   ` Jason
2014-01-12 17:13 ` [PATCH 2/6] ui-snapshot: set unused cgit_filter fields to zero john
2014-01-12 17:13 ` [PATCH 3/6] filter: pass extra arguments via cgit_open_filter john
2014-01-12 19:20   ` Jason
2014-01-12 17:13 ` [PATCH 4/6] filter: add fprintf_filter function john
2014-01-12 19:23   ` Jason
2014-01-12 19:35     ` john
2014-01-12 19:40       ` Jason
2014-01-12 17:13 ` [PATCH 5/6] filter: add interface layer john
2014-01-12 17:13 ` [PATCH 6/6] filter: introduce "filter type" prefix john

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1389546691.git.john@keeping.me.uk \
    --to=cgit@lists.zx2c4.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).