some tiny notes..  

- we have ibstr  

- please use a strbuf to construct the URL  

- please use the HKEY macro where initialising strings instead of strlen  

there is...StrBufUrlescAppend; StrBufHexescAppend, (maybe you want to create
a base64 wrapper instead?)  

- use StrBufAppendTemplate() to add your result to target; it offers the
standard escaping methods to the token (see next point)  

- you probably don't want <?MAIL:SUMM:SUBJECT("X")> in the URL but rather "U"
 
>  So Nov 21 2010 23:27:37 EST von  "ajc" <a...@uncensored.citadel.org> 
>Betreff: [SCM] citadel.org branch, master, updated.
>81ad90568d8c00c2104339d2b132ba2945bb01f9
>
>  
> Push to the project "citadel.org ": The branch, master has been updated
> via  81ad90568d8c00c2104339d2b132ba2945bb01f9 (commit)
> from  36821734e42c9200de69079d314b71c5a6b5526e (commit)
> 
> Those revisions listed above that are new to this repository have not
>appeared on any other notification email; so we list those revisions in full,
>below.
> - Log -----------------------------------------------------------------
>
>http://code.citadel.org/cgit.cgi/git.citadel.org/commit/?id=81ad90568d8c00c210
>4339d2b132ba2945bb01f9
> commit 81ad90568d8c00c2104339d2b132ba2945bb01f9
> Author: Art Cancro <a...@citadel.org>
> Date:   Sun Nov 21 23:27:11 2010 -0500
> 
> Began working on a permalink system WARNING NOT WORKING YET
> 
> -----------------------------------------------------------------------
> 
> Summary of changes:
> webcit/blogview_renderer.c          |   23 +++++++++++++++++++++++
> webcit/static/styles/webcit.css     |    9 +++++++++
> webcit/static/t/view_blog_post.html |    2 +-
> 3 files changed, 33 insertions(+), 1 deletions(-)
> 
> 
> hooks/post-receive
> -- 
> citadel.org 
> 
>
>
>  

  

 

Reply via email to