Show list of modified files in ui-commit.c

Compare current commit with 1.parent, and for each affected file display
current filemode, old filemode if changed, current filename and source
filename if it was a copy/rename.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>

5 files changed, 281 insertions, 8 deletions

diff --git a/cgit.css b/cgit.css
index 94372d6..ae304fa 100644
--- a/cgit.css
+++ b/cgit.css
@@ -1,131 +1,166 @@
 body {
         font-family: arial;
         font-size: normal;
         background: white;
         padding: 0em;
-        margin: 0.5em;
+        margin: 0.5em 1em;
 }
 
 
 h2 {
         font-size: normal;
         font-weight: bold;
         margin-bottom: 0.1em;
 }
 
 a {
         color: blue;
         text-decoration: none;
 }
 
 a:hover {
         text-decoration: underline;
 }
 
 table.list {
         border: solid 1px black;
         border-collapse: collapse;
         border: solid 1px #aaa;
 }
 table.list tr {
         background: white;
 }
 table.list tr:hover {
         background: #eee;
 }
 table.list th {
         font-weight: bold;
         background: #ddd;
         border-bottom: solid 1px #aaa;
         padding: 0.1em 0.5em 0.1em 0.5em;
         vertical-align: baseline;
 }
 table.list td {
         border: none;
         padding: 0.1em 0.5em 0.1em 0.5em;
 }
 img {
         border: none;
 }
 
 
 div#header {
-        background-color: #ddd;
+        background-color: #eee;
         padding: 0.25em 0.25em 0.25em 0.5em;
         font-size: 150%;
         font-weight: bold;
-        border: solid 1px #aaa;
+        border: solid 1px #ccc;
         vertical-align: middle;
 }
 div#header img#logo {
         float: right;
 }
+div#header a {
+        color: black;
+}
 
 div#content {
         margin: 0.5em 0.5em;
 }
 
 div#blob {
         border: solid 1px black;
 }
 
 div.error {
         color: red;
         font-weight: bold;
         margin: 1em 2em;
 }
 div.ls-dir a {
         font-weight: bold;
 }
 th.filesize, td.filesize {
         text-align: right;
 }
 th.filemode, td.filemode {
         text-align: center;
 }
 
 td.blob {
         white-space: pre;
         font-family: courier;
         font-size: 100%;
         background-color: white;
 }
 
 table.log td {
         white-space: nowrap;
 }
 
 table.commit-info {
         border-collapse: collapse;
-        margin-top: 1em;
-        
+        margin-top: 1.5em;
 }
 table.commit-info th {
         text-align: left;
         font-weight: normal;
         padding: 0.1em 1em 0.1em 0.1em;
 }
 table.commit-info td {
         font-weight: normal;
         padding: 0.1em 1em 0.1em 0.1em;
 }
 div.commit-subject {
         font-weight: bold;
-        font-size: 110%;
-        margin: 1em 0em 1em;
+        font-size: 125%;
+        margin: 1.5em 0em 0.5em 0em;
+        padding: 0em;
 }
 div.commit-msg {
         white-space: pre;
-        font-family: courier;
+        font-family: monospace;
+}
+table.diffstat {
+        border-collapse: collapse;
+        margin-top: 1.5em;
+}
+table.diffstat th {
+        font-weight: normal;
+        text-align: left;
+        text-decoration: underline;
+        padding: 0.1em 1em 0.1em 0.1em;
         font-size: 100%;
 }
+table.diffstat td {
+        padding: 0.1em 1em 0.1em 0.1em;
+        font-size: 100%;
+}
+table.diffstat td span.modechange {
+        padding-left: 1em;
+        color: red;
+}
+table.diffstat td.add a {
+        color: green;
+}
+table.diffstat td.del a {
+        color: red;
+}
+table.diffstat td.upd a {
+        color: blue;
+}
+table.diffstat td.summary {
+        /* border-top: solid 1px black; */
+        color: #888;
+        padding-top: 0.5em;
+}
 .sha1 {
         font-family: courier;
         font-size: 90%;
 }
 .left {
         text-align: left;
 }
 .right {
         text-align: right;
 }
diff --git a/cgit.h b/cgit.h
index 02cc26a..37584d6 100644
--- a/cgit.h
+++ b/cgit.h
@@ -1,107 +1,108 @@
 #ifndef CGIT_H
 #define CGIT_H
 
 #include "git.h"
 #include <openssl/sha.h>
 #include <ctype.h>
 #include <sched.h>
 
 typedef void (*configfn)(const char *name, const char *value);
 
 struct cacheitem {
         char *name;
         struct stat st;
         int ttl;
         int fd;
 };
 
 struct commitinfo {
         struct commit *commit;
         char *author;
         char *author_email;
         unsigned long author_date;
         char *committer;
         char *committer_email;
         unsigned long committer_date;
         char *subject;
         char *msg;
 };
 
 extern const char cgit_version[];
 
 extern char *cgit_root;
 extern char *cgit_root_title;
 extern char *cgit_css;
 extern char *cgit_logo;
 extern char *cgit_logo_link;
 extern char *cgit_virtual_root;
 extern char *cgit_cache_root;
 
 extern int cgit_nocache;
 extern int cgit_max_lock_attempts;
 extern int cgit_cache_root_ttl;
 extern int cgit_cache_repo_ttl;
 extern int cgit_cache_dynamic_ttl;
 extern int cgit_cache_static_ttl;
 extern int cgit_cache_max_create_time;
 
 extern char *cgit_repo_name;
 extern char *cgit_repo_desc;
 extern char *cgit_repo_owner;
 
 extern int cgit_query_has_symref;
 extern int cgit_query_has_sha1;
 
 extern char *cgit_querystring;
 extern char *cgit_query_repo;
 extern char *cgit_query_page;
 extern char *cgit_query_head;
 extern char *cgit_query_sha1;
 extern int   cgit_query_ofs;
 
 extern int htmlfd;
 
 extern void cgit_global_config_cb(const char *name, const char *value);
 extern void cgit_repo_config_cb(const char *name, const char *value);
 extern void cgit_querystring_cb(const char *name, const char *value);
 
 extern void *cgit_free_commitinfo(struct commitinfo *info);
 
 extern char *fmt(const char *format,...);
 
 extern void html(const char *txt);
 extern void htmlf(const char *format,...);
 extern void html_txt(char *txt);
 extern void html_attr(char *txt);
 extern void html_link_open(char *url, char *title, char *class);
 extern void html_link_close(void);
+extern void html_filemode(unsigned short mode);
 
 extern int cgit_read_config(const char *filename, configfn fn);
 extern int cgit_parse_query(char *txt, configfn fn);
 extern struct commitinfo *cgit_parse_commit(struct commit *commit);
 
 extern void cache_prepare(struct cacheitem *item);
 extern int cache_lock(struct cacheitem *item);
 extern int cache_unlock(struct cacheitem *item);
 extern int cache_cancel_lock(struct cacheitem *item);
 extern int cache_exist(struct cacheitem *item);
 extern int cache_expired(struct cacheitem *item);
 
 extern char *cgit_repourl(const char *reponame);
 extern char *cgit_pageurl(const char *reponame, const char *pagename, 
                           const char *query);
 
 extern void cgit_print_error(char *msg);
 extern void cgit_print_date(unsigned long secs);
 extern void cgit_print_docstart(char *title, struct cacheitem *item);
 extern void cgit_print_docend();
 extern void cgit_print_pageheader(char *title);
 
 extern void cgit_print_repolist(struct cacheitem *item);
 extern void cgit_print_summary();
 extern void cgit_print_log(const char *tip, int ofs, int cnt);
 extern void cgit_print_view(const char *hex);
 extern void cgit_print_tree(const char *hex);
 extern void cgit_print_commit(const char *hex);
 
 #endif /* CGIT_H */
diff --git a/git.h b/git.h
index a3f977c..922a167 100644
--- a/git.h
+++ b/git.h
@@ -1,462 +1,568 @@
 #ifndef GIT_H
 #define GIT_H
 
 
 /*
  * from git:git-compat-util.h 
  */
 
 
 #ifndef FLEX_ARRAY
 #if defined(__GNUC__) && (__GNUC__ < 3)
 #define FLEX_ARRAY 0
 #else
 #define FLEX_ARRAY /* empty */
 #endif
 #endif
 
 
 #include <unistd.h>
 #include <stdio.h>
 #include <sys/stat.h>
 #include <fcntl.h>
 #include <stddef.h>
 #include <stdlib.h>
 #include <stdarg.h>
 #include <string.h>
 #include <errno.h>
 #include <limits.h>
 #include <sys/param.h>
 #include <netinet/in.h>
 #include <sys/types.h>
 #include <dirent.h>
 #include <time.h>
 
 
 /* On most systems <limits.h> would have given us this, but
  * not on some systems (e.g. GNU/Hurd).
  */
 #ifndef PATH_MAX
 #define PATH_MAX 4096
 #endif
 
 #ifdef __GNUC__
 #define NORETURN __attribute__((__noreturn__))
 #else
 #define NORETURN
 #ifndef __attribute__
 #define __attribute__(x)
 #endif
 #endif
 
 
 extern void die(const char *err, ...) NORETURN __attribute__((format (printf, 1, 2)));
 
 
 static inline char* xstrdup(const char *str)
 {
         char *ret = strdup(str);
         if (!ret)
                 die("Out of memory, strdup failed");
         return ret;
 }
 
 static inline void *xmalloc(size_t size)
 {
         void *ret = malloc(size);
         if (!ret && !size)
                 ret = malloc(1);
         if (!ret)
                 die("Out of memory, malloc failed");
 #ifdef XMALLOC_POISON
         memset(ret, 0xA5, size);
 #endif
         return ret;
 }
 
 static inline void *xrealloc(void *ptr, size_t size)
 {
         void *ret = realloc(ptr, size);
         if (!ret && !size)
                 ret = realloc(ptr, 1);
         if (!ret)
                 die("Out of memory, realloc failed");
         return ret;
 }
 
 static inline void *xcalloc(size_t nmemb, size_t size)
 {
         void *ret = calloc(nmemb, size);
         if (!ret && (!nmemb || !size))
                 ret = calloc(1, 1);
         if (!ret)
                 die("Out of memory, calloc failed");
         return ret;
 }
 
 static inline ssize_t xread(int fd, void *buf, size_t len)
 {
         ssize_t nr;
         while (1) {
                 nr = read(fd, buf, len);
                 if ((nr < 0) && (errno == EAGAIN || errno == EINTR))
                         continue;
                 return nr;
         }
 }
 
 static inline ssize_t xwrite(int fd, const void *buf, size_t len)
 {
         ssize_t nr;
         while (1) {
                 nr = write(fd, buf, len);
                 if ((nr < 0) && (errno == EAGAIN || errno == EINTR))
                         continue;
                 return nr;
         }
 }
 
 
 
 
 /*
  * from git:cache.h
  */
 
 
 /* Convert to/from hex/sha1 representation */
 #define MINIMUM_ABBREV 4
 #define DEFAULT_ABBREV 7
 
+extern const unsigned char null_sha1[20];
+
 extern int sha1_object_info(const unsigned char *, char *, unsigned long *);
 
 extern void * read_sha1_file(const unsigned char *sha1, char *type, unsigned long *size);
 
 extern int get_sha1(const char *str, unsigned char *sha1);
 extern int get_sha1_hex(const char *hex, unsigned char *sha1);
         extern char *sha1_to_hex(const unsigned char *sha1);/* static buffer result! */
 
+static inline int is_null_sha1(const unsigned char *sha1)
+{
+        return !memcmp(sha1, null_sha1, 20);
+}
+static inline int hashcmp(const unsigned char *sha1, const unsigned char *sha2)
+{
+        return memcmp(sha1, sha2, 20);
+}
+static inline void hashcpy(unsigned char *sha_dst, const unsigned char *sha_src)
+{
+        memcpy(sha_dst, sha_src, 20);
+}
+static inline void hashclr(unsigned char *hash)
+{
+        memset(hash, 0, 20);
+}
+
+
 
 
 /*
  * from git:object.h 
  */
 
 struct object_list {
         struct object *item;
         struct object_list *next;
 };
 
 struct object_refs {
         unsigned count;
         struct object *base;
         struct object *ref[FLEX_ARRAY]; /* more */
 };
 
 struct object_array {
         unsigned int nr;
         unsigned int alloc;
         struct object_array_entry {
                 struct object *item;
                 const char *name;
         } *objects;
 };
 
 #define TYPE_BITS   3
 #define FLAG_BITS  27
 
 /*
  * The object type is stored in 3 bits.
  */
 struct object {
         unsigned parsed : 1;
         unsigned used : 1;
         unsigned type : TYPE_BITS;
         unsigned flags : FLAG_BITS;
         unsigned char sha1[20];
 };
 
 
 /*
  * from git:tree.h
  */
 
 struct tree {
         struct object object;
         void *buffer;
         unsigned long size;
 };
 
 
 struct tree *lookup_tree(const unsigned char *sha1);
 int parse_tree_buffer(struct tree *item, void *buffer, unsigned long size);
 int parse_tree(struct tree *tree);
 struct tree *parse_tree_indirect(const unsigned char *sha1);
 
 typedef int (*read_tree_fn_t)(const unsigned char *, const char *, int, const char *, unsigned int, int);
 
 extern int read_tree_recursive(struct tree *tree,
                                const char *base, int baselen,
                                int stage, const char **match,
                                read_tree_fn_t fn);
 
 extern int read_tree(struct tree *tree, int stage, const char **paths);
 
 
 /* from git:commit.h */
 
 struct commit_list {
         struct commit *item;
         struct commit_list *next;
 };
 
 struct commit {
         struct object object;
         void *util;
         unsigned long date;
         struct commit_list *parents;
         struct tree *tree;
         char *buffer;
 };
 
 
 struct commit *lookup_commit(const unsigned char *sha1);
 struct commit *lookup_commit_reference(const unsigned char *sha1);
 struct commit *lookup_commit_reference_gently(const unsigned char *sha1,
                                               int quiet);
 
 int parse_commit_buffer(struct commit *item, void *buffer, unsigned long size);
 int parse_commit(struct commit *item);
 
 struct commit_list * commit_list_insert(struct commit *item, struct commit_list **list_p);
 struct commit_list * insert_by_date(struct commit *item, struct commit_list **list);
 
 void free_commit_list(struct commit_list *list);
 
 void sort_by_date(struct commit_list **list);
 
 /* Commit formats */
 enum cmit_fmt {
         CMIT_FMT_RAW,
         CMIT_FMT_MEDIUM,
         CMIT_FMT_DEFAULT = CMIT_FMT_MEDIUM,
         CMIT_FMT_SHORT,
         CMIT_FMT_FULL,
         CMIT_FMT_FULLER,
         CMIT_FMT_ONELINE,
         CMIT_FMT_EMAIL,
 
         CMIT_FMT_UNSPECIFIED,
 };
 
 extern unsigned long pretty_print_commit(enum cmit_fmt fmt, const struct commit *, unsigned long len, char *buf, unsigned long space, int abbrev, const char *subject, const char *after_subject, int relative_date);
 
 
 typedef void (*topo_sort_set_fn_t)(struct commit*, void *data);
 typedef void* (*topo_sort_get_fn_t)(struct commit*);
 
 
 
+/*
+ * from git:diffcore.h
+ */
+
+struct diff_filespec {
+        unsigned char sha1[20];
+        char *path;
+        void *data;
+        void *cnt_data;
+        unsigned long size;
+                int xfrm_flags;          /* for use by the xfrm */
+                unsigned short mode; /* file mode */
+        unsigned sha1_valid : 1; /* if true, use sha1 and trust mode;
+                                  * if false, use the name and read from
+                                  * the filesystem.
+                                  */
+#define DIFF_FILE_VALID(spec) (((spec)->mode) != 0)
+        unsigned should_free : 1; /* data should be free()'ed */
+        unsigned should_munmap : 1; /* data should be munmap()'ed */
+};
+
+struct diff_filepair {
+        struct diff_filespec *one;
+        struct diff_filespec *two;
+        unsigned short int score;
+        char status; /* M C R N D U (see Documentation/diff-format.txt) */
+        unsigned source_stays : 1; /* all of R/C are copies */
+        unsigned broken_pair : 1;
+        unsigned renamed_pair : 1;
+};
+
+#define DIFF_PAIR_UNMERGED(p) \
+        (!DIFF_FILE_VALID((p)->one) && !DIFF_FILE_VALID((p)->two))
+
+#define DIFF_PAIR_RENAME(p) ((p)->renamed_pair)
+
+#define DIFF_PAIR_BROKEN(p) \
+        ( (!DIFF_FILE_VALID((p)->one) != !DIFF_FILE_VALID((p)->two)) && \
+          ((p)->broken_pair != 0) )
+
+#define DIFF_PAIR_TYPE_CHANGED(p) \
+        ((S_IFMT & (p)->one->mode) != (S_IFMT & (p)->two->mode))
+
+#define DIFF_PAIR_MODE_CHANGED(p) ((p)->one->mode != (p)->two->mode)
+
+extern void diff_free_filepair(struct diff_filepair *);
+
+extern int diff_unmodified_pair(struct diff_filepair *);
+
+struct diff_queue_struct {
+        struct diff_filepair **queue;
+        int alloc;
+        int nr;
+};
+
 
 /*
  *  from git:diff.h
  */
 
 
 struct rev_info;
 struct diff_options;
 struct diff_queue_struct;
 
 typedef void (*change_fn_t)(struct diff_options *options,
                  unsigned old_mode, unsigned new_mode,
                  const unsigned char *old_sha1,
                  const unsigned char *new_sha1,
                  const char *base, const char *path);
 
 typedef void (*add_remove_fn_t)(struct diff_options *options,
                     int addremove, unsigned mode,
                     const unsigned char *sha1,
                     const char *base, const char *path);
 
 typedef void (*diff_format_fn_t)(struct diff_queue_struct *q,
                 struct diff_options *options, void *data);
 
         #define DIFF_FORMAT_RAW         0x0001
         #define DIFF_FORMAT_DIFFSTAT0x0002
         #define DIFF_FORMAT_NUMSTAT0x0004
         #define DIFF_FORMAT_SUMMARY0x0008
         #define DIFF_FORMAT_PATCH0x0010
 
 /* These override all above */
         #define DIFF_FORMAT_NAME0x0100
         #define DIFF_FORMAT_NAME_STATUS0x0200
         #define DIFF_FORMAT_CHECKDIFF0x0400
 
 /* Same as output_format = 0 but we know that -s flag was given
  * and we should not give default value to output_format.
  */
         #define DIFF_FORMAT_NO_OUTPUT0x0800
 
         #define DIFF_FORMAT_CALLBACK0x1000
 
 struct diff_options {
         const char *filter;
         const char *orderfile;
         const char *pickaxe;
         const char *single_follow;
         unsigned recursive:1,
                  tree_in_recursive:1,
                  binary:1,
                  text:1,
                  full_index:1,
                  silent_on_remove:1,
                  find_copies_harder:1,
                  color_diff:1,
                  color_diff_words:1;
         int context;
         int break_opt;
         int detect_rename;
         int line_termination;
         int output_format;
         int pickaxe_opts;
         int rename_score;
         int reverse_diff;
         int rename_limit;
         int setup;
         int abbrev;
         const char *msg_sep;
         const char *stat_sep;
         long xdl_opts;
 
         int stat_width;
         int stat_name_width;
 
         int nr_paths;
         const char **paths;
         int *pathlens;
         change_fn_t change;
         add_remove_fn_t add_remove;
         diff_format_fn_t format_callback;
         void *format_callback_data;
 };
 
 enum color_diff {
         DIFF_RESET = 0,
         DIFF_PLAIN = 1,
         DIFF_METAINFO = 2,
         DIFF_FRAGINFO = 3,
         DIFF_FILE_OLD = 4,
         DIFF_FILE_NEW = 5,
         DIFF_COMMIT = 6,
         DIFF_WHITESPACE = 7,
 };
 
 
+extern int diff_tree_sha1(const unsigned char *old, const unsigned char *new,
+                          const char *base, struct diff_options *opt);
+
+extern int diff_root_tree_sha1(const unsigned char *new, const char *base,
+                               struct diff_options *opt);
+
+extern int git_diff_ui_config(const char *var, const char *value);
+extern void diff_setup(struct diff_options *);
+extern int diff_opt_parse(struct diff_options *, const char **, int);
+extern int diff_setup_done(struct diff_options *);
+
+
+extern void diffcore_std(struct diff_options *);
+extern void diff_flush(struct diff_options*);
+
+
+/* diff-raw status letters */
+        #define DIFF_STATUS_ADDED       'A'
+        #define DIFF_STATUS_COPIED      'C'
+        #define DIFF_STATUS_DELETED     'D'
+        #define DIFF_STATUS_MODIFIED    'M'
+        #define DIFF_STATUS_RENAMED     'R'
+        #define DIFF_STATUS_TYPE_CHANGED'T'
+        #define DIFF_STATUS_UNKNOWN     'X'
+        #define DIFF_STATUS_UNMERGED    'U'
+
 
 
 /*
  * from git:refs.g
  */
 
 typedef int each_ref_fn(const char *refname, const unsigned char *sha1, int flags, void *cb_data);
 extern int head_ref(each_ref_fn, void *);
 extern int for_each_ref(each_ref_fn, void *);
 extern int for_each_tag_ref(each_ref_fn, void *);
 extern int for_each_branch_ref(each_ref_fn, void *);
 extern int for_each_remote_ref(each_ref_fn, void *);
 
 
 
 /*
  * from git:revision.h
  */
 
 struct rev_info;
 struct log_info;
 
 typedef void (prune_fn_t)(struct rev_info *revs, struct commit *commit);
 
 struct rev_info {
         /* Starting list */
         struct commit_list *commits;
         struct object_array pending;
 
         /* Basic information */
         const char *prefix;
         void *prune_data;
         prune_fn_t *prune_fn;
 
         /* Traversal flags */
                 unsigned intdense:1,
                         no_merges:1,
                         no_walk:1,
                         remove_empty_trees:1,
                         simplify_history:1,
                         lifo:1,
                         topo_order:1,
                         tag_objects:1,
                         tree_objects:1,
                         blob_objects:1,
                         edge_hint:1,
                         limited:1,
                         unpacked:1, /* see also ignore_packed below */
                         boundary:1,
                         parents:1;
 
         /* Diff flags */
                 unsigned intdiff:1,
                         full_diff:1,
                         show_root_diff:1,
                         no_commit_id:1,
                         verbose_header:1,
                         ignore_merges:1,
                         combine_merges:1,
                         dense_combined_merges:1,
                         always_show_header:1;
 
         /* Format info */
                 unsigned intshown_one:1,
                         abbrev_commit:1,
                         relative_date:1;
 
         const char **ignore_packed; /* pretend objects in these are unpacked */
         int num_ignore_packed;
 
                 unsigned intabbrev;
                 enum cmit_fmtcommit_format;
         struct log_info *loginfo;
                 int     nr, total;
                 const char*mime_boundary;
                 const char*message_id;
                 const char*ref_message_id;
                 const char*add_signoff;
                 const char*extra_headers;
 
         /* Filter by commit log message */
                 struct grep_opt*grep_filter;
 
         /* special limits */
         int max_count;
         unsigned long max_age;
         unsigned long min_age;
 
         /* diff info for patches and for paths limiting */
         struct diff_options diffopt;
         struct diff_options pruning;
 
         topo_sort_set_fn_t topo_setter;
         topo_sort_get_fn_t topo_getter;
 };
 
 
 extern void init_revisions(struct rev_info *revs, const char *prefix);
 extern int setup_revisions(int argc, const char **argv, struct rev_info *revs, const char *def);
 extern int handle_revision_arg(const char *arg, struct rev_info *revs,int flags,int cant_be_filename);
 
 extern void prepare_revision_walk(struct rev_info *revs);
 extern struct commit *get_revision(struct rev_info *revs);
 
 
 
+/* from git:log-tree.h */
+
+int log_tree_commit(struct rev_info *, struct commit *);
+
+
 
 #endif /* GIT_H */
diff --git a/html.c b/html.c
index fab67bd..8a69659 100644
--- a/html.c
+++ b/html.c
@@ -1,110 +1,130 @@
 /* html.c: helper functions for html output
  *
  * Copyright (C) 2006 Lars Hjemli
  *
  * Licensed under GNU General Public License v2
  *   (see COPYING for full license text)
  */
 
 #include "cgit.h"
 
 char *fmt(const char *format, ...)
 {
         static char buf[8][1024];
         static int bufidx;
         int len;
         va_list args;
 
         bufidx++;
         bufidx &= 7;
 
         va_start(args, format);
         len = vsnprintf(buf[bufidx], sizeof(buf[bufidx]), format, args);
         va_end(args);
         if (len>sizeof(buf[bufidx]))
                 die("[html.c] string truncated: %s", format);
         return buf[bufidx];
 }
 
 void html(const char *txt)
 {
         write(htmlfd, txt, strlen(txt));
 }
 
 void htmlf(const char *format, ...)
 {
         static char buf[65536];
         va_list args;
 
         va_start(args, format);
         vsnprintf(buf, sizeof(buf), format, args);
         va_end(args);
         html(buf);
 }
 
 void html_txt(char *txt)
 {
         char *t = txt;
         while(*t){
                 int c = *t;
                 if (c=='<' || c=='>' || c=='&') {
                         *t = '\0';
                         html(txt);
                         *t = c;
                         if (c=='>')
                                 html("&gt;");
                         else if (c=='<')
                                 html("&lt;");
                         else if (c=='&')
                                 html("&amp;");
                         txt = t+1;
                 }
                 t++;
         }
         if (t!=txt)
                 html(txt);
 }
 
 
 void html_attr(char *txt)
 {
         char *t = txt;
         while(*t){
                 int c = *t;
                 if (c=='<' || c=='>' || c=='\'') {
                         *t = '\0';
                         html(txt);
                         *t = c;
                         if (c=='>')
                                 html("&gt;");
                         else if (c=='<')
                                 html("&lt;");
                         else if (c=='\'')
                                 html("&quote;");
                         txt = t+1;
                 }
                 t++;
         }
         if (t!=txt)
                 html(txt);
 }
 
 void html_link_open(char *url, char *title, char *class)
 {
         html("<a href='");
         html_attr(url);
         if (title) {
                 html("' title='");
                 html_attr(title);
         }
         if (class) {
                 html("' class='");
                 html_attr(class);
         }
         html("'>");
 }
 
 void html_link_close(void)
 {
         html("</a>");
 }
+
+void html_fileperm(unsigned short mode)
+{
+        htmlf("%c%c%c", (mode & 4 ? 'r' : '-'), 
+              (mode & 2 ? 'w' : '-'), (mode & 1 ? 'x' : '-'));
+}
+
+void html_filemode(unsigned short mode)
+{
+        if (S_ISDIR(mode))
+                html("d");
+        else if (S_ISLNK(mode))
+                html("l");
+        else
+                html("-");
+        html_fileperm(mode >> 6);
+        html_fileperm(mode >> 3);
+        html_fileperm(mode);
+}
+
diff --git a/ui-commit.c b/ui-commit.c
index 8916212..c5ee8e7 100644
--- a/ui-commit.c
+++ b/ui-commit.c
@@ -1,66 +1,177 @@
 /* ui-commit.c: generate commit view
  *
  * Copyright (C) 2006 Lars Hjemli
  *
  * Licensed under GNU General Public License v2
  *   (see COPYING for full license text)
  */
 
 #include "cgit.h"
 
+int files = 0;
+
+void print_filepair(struct diff_filepair *pair)
+{
+        char *query;
+        char *class;
+        
+        switch (pair->status) {
+        case DIFF_STATUS_ADDED:
+                class = "add";
+                break;
+        case DIFF_STATUS_COPIED:
+                class = "cpy";
+                break;
+        case DIFF_STATUS_DELETED:
+                class = "del";
+                break;
+        case DIFF_STATUS_MODIFIED:
+                class = "upd";
+                break;
+        case DIFF_STATUS_RENAMED:
+                class = "mov";
+                break;
+        case DIFF_STATUS_TYPE_CHANGED:
+                class = "typ";
+                break;
+        case DIFF_STATUS_UNKNOWN:
+                class = "unk";
+                break;
+        case DIFF_STATUS_UNMERGED:
+                class = "stg";
+                break;
+        default:
+                die("bug: unhandled diff status %c", pair->status);
+        }
+
+        html("<tr>");
+        htmlf("<td class='mode'>");
+        html_filemode(pair->two->mode);
+        if (pair->one->mode != pair->two->mode) {
+                html("<span class='modechange'>[");
+                html_filemode(pair->one->mode);
+                html("]</span>");
+        }
+        htmlf("</td><td class='%s'>", class);
+                query = fmt("id=%s", sha1_to_hex(pair->two->sha1));
+        html_link_open(cgit_pageurl(cgit_query_repo, "view", query), 
+                       NULL, NULL);
+        if (pair->status == DIFF_STATUS_COPIED || 
+            pair->status == DIFF_STATUS_RENAMED) {
+                html_txt(pair->two->path);
+                htmlf("</a> (%s from ", pair->status == DIFF_STATUS_COPIED ? 
+                      "copied" : "renamed");
+                        query = fmt("id=%s", sha1_to_hex(pair->one->sha1));
+                html_link_open(cgit_pageurl(cgit_query_repo, "view", query), 
+                               NULL, NULL);
+                html_txt(pair->one->path);
+                html("</a>)");
+        } else {
+                html_txt(pair->two->path);
+                html("</a>");
+        }
+        html("<td>");
+
+        //TODO: diffstat graph
+        
+        html("</td></tr>\n");
+                files++;
+}
+
+void diff_format_cb(struct diff_queue_struct *q,
+                    struct diff_options *options, void *data)
+{
+        int i;
+
+        for (i = 0; i < q->nr; i++) {
+                if (q->queue[i]->status == 'U')
+                        continue;
+                print_filepair(q->queue[i]);
+        }
+}
+
+void cgit_diffstat(struct commit *commit)
+{
+        struct diff_options opt;
+        int ret;
+
+        diff_setup(&opt);
+        opt.output_format = DIFF_FORMAT_CALLBACK;
+        opt.detect_rename = 1;
+        opt.recursive = 1;
+        opt.format_callback = diff_format_cb;
+        diff_setup_done(&opt);
+        
+        if (commit->parents)
+                ret = diff_tree_sha1(commit->parents->item->object.sha1,
+                                     commit->object.sha1,
+                                     "", &opt);
+        else
+                ret = diff_root_tree_sha1(commit->object.sha1, "", &opt);
+
+        diffcore_std(&opt);
+        diff_flush(&opt);
+}
+
 void cgit_print_commit(const char *hex)
 {
         struct commit *commit;
         struct commitinfo *info;
         struct commit_list *p;
         unsigned char sha1[20];
         char *query;
 
         if (get_sha1(hex, sha1)) {
                 cgit_print_error(fmt("Bad object id: %s", hex));
                 return;
         }
         commit = lookup_commit_reference(sha1);
         if (!commit) {
                 cgit_print_error(fmt("Bad commit reference: %s", hex));
                 return;
         }
         info = cgit_parse_commit(commit);
 
         html("<table class='commit-info'>\n");
         html("<tr><th>author</th><td>");
         html_txt(info->author);
         html(" ");
         html_txt(info->author_email);
         html("</td><td class='right'>");
         cgit_print_date(info->author_date);
         html("</td></tr>\n");
         html("<tr><th>committer</th><td>");
         html_txt(info->committer);
         html(" ");
         html_txt(info->committer_email);
         html("</td><td class='right'>");
         cgit_print_date(info->committer_date);
         html("</td></tr>\n");
         html("<tr><th>tree</th><td colspan='2' class='sha1'><a href='");
         query = fmt("id=%s", sha1_to_hex(commit->tree->object.sha1));
         html_attr(cgit_pageurl(cgit_query_repo, "tree", query));
         htmlf("'>%s</a></td></tr>\n", sha1_to_hex(commit->tree->object.sha1));
               for (p = commit->parents; p ; p = p->next) {
                 html("<tr><th>parent</th>"
                      "<td colspan='2' class='sha1'>"
                      "<a href='");
                 query = fmt("id=%s", sha1_to_hex(p->item->object.sha1));
                 html_attr(cgit_pageurl(cgit_query_repo, "commit", query));
                 htmlf("'>%s</a></td></tr>\n", 
                       sha1_to_hex(p->item->object.sha1));
         }
         html("</table>\n");
         html("<div class='commit-subject'>");
         html_txt(info->subject);
         html("</div>");
         html("<div class='commit-msg'>");
         html_txt(info->msg);
         html("</div>");
+        html("<table class='diffstat'>");
+        html("<tr><th colspan='3'>Affected files</tr>\n");
+        cgit_diffstat(commit);
+        htmlf("<tr><td colspan='3' class='summary'>"
+              "%d file%s changed</td></tr>\n", files, files > 1 ? "s" : "");
+        html("</table>");
         cgit_free_commitinfo(info);
 }