Cleanup, types and compiler warnings
authorChristian Thaeter <ct@pipapo.org>
Wed, 16 Jan 2008 13:10:54 +0000 (14:10 +0100)
committerChristian Thaeter <ct@pipapo.org>
Wed, 16 Jan 2008 13:10:54 +0000 (14:10 +0100)
src/actions.c
src/log.c
src/options.c
src/query.c
src/repo.c
src/webgit.c
src/webgit.h

index 6c17d7c..c5a4ce1 100644 (file)
@@ -241,7 +241,7 @@ webgit_raw_action (struct webgit_query* query)
 
   buf = read_object_with_reference (sha1, "blob", &size, NULL);
 
-  free (query->content_type);
+  free ((char*)query->content_type);
   query->content_type = webgit_mimetype (query->path);
 
   return html_binary (buf, size);
index 62560ed..0a27712 100644 (file)
--- a/src/log.c
+++ b/src/log.c
@@ -45,10 +45,10 @@ webgit_log_link (struct webgit_query* query,
                                                            head?head:"master"),
                                                  object ? html_fmt("&object=%.*s",
                                                                    object_len, object)
-                                                 : "",
+                                                 : html (),
                                                  count > 0 ? html_fmt("&count=%d",
                                                                       count)
-                                                 : ""
+                                                 : html ()
                                                  )
                                     )
                          ),
index 59b499e..cd3f582 100644 (file)
@@ -20,6 +20,7 @@
 */
 
 #include "options.h"
+#include "query.h"
 
 #include <stdlib.h>
 #include <stdio.h>
@@ -28,6 +29,7 @@
 static int
 conf_query_opt (struct webgit_query* query, char* arg)
 {
+  (void) query;
   if (arg)
     setenv ("QUERY_STRING", arg, 1);
   else
@@ -38,6 +40,7 @@ conf_query_opt (struct webgit_query* query, char* arg)
 static int
 conf_method_opt (struct webgit_query* query, char* arg)
 {
+  (void) query;
   if (arg)
     setenv ("REQUEST_METHOD", arg, 1);
   else
@@ -49,7 +52,6 @@ static int
 conf_repo_opt (struct webgit_query* query, char* arg)
 {
   webgit_query_add_repo (query, arg);
-
   return 0;
 }
 
index 257cbe8..a2a4059 100644 (file)
@@ -86,7 +86,7 @@ webgit_query_add_repo (struct webgit_query* q, const char* path)
   llist_insert_next (p, &ri->node);
 }
 
-const int
+int
 webgit_validate_string (const char *s, size_t v_sz)
 {
   for (const char* c = ";$&|<>!`#"; *c; ++c)
index feb3234..94faf22 100644 (file)
@@ -24,6 +24,7 @@
 
 #define SHA1_HEADER <openssl/sha.h>
 #include "git/cache.h"
+#include "git/refs.h"
 
 #include <sys/types.h>
 #include <sys/stat.h>
@@ -207,7 +208,7 @@ find_last (const char *refname, const unsigned char *sha1, int flags, void *ri)
 struct webgit_repo_info*
 webgit_repo_enter (struct webgit_query* query)
 {
-  struct webgit_repo_info* ri;
+  struct webgit_repo_info* ri = NULL;
   LLIST_FOREACH (&query->repos, node)
     {
       ri = (struct webgit_repo_info*) node;
@@ -320,7 +321,7 @@ webgit_repoinfo_new (struct webgit_query* query, const char* path)
 void
 webgit_repoinfo_find_last (struct webgit_repo_info* self)
 {
-  if (self->age != ~0)
+  if (self->age != ~0UL)
     return;
 
   chdir (self->path);
index 390d6cf..9caf85a 100644 (file)
@@ -81,7 +81,7 @@ webgit_email_link (Html name, Html email)
 }
 
 
-const char*
+char*
 webgit_mimetype (const char* name)
 {
   char buf[256];
index 0fa5202..08858fc 100644 (file)
@@ -34,7 +34,7 @@ struct webgit_query
 
   int deref_to_tree;
 
-  const char* content_type;
+  char* content_type;
 
   /* confiuration parameters */
   int count_def;
@@ -83,7 +83,7 @@ struct webgit_repo_info
 Html
 webgit_email_link (Html name, Html email);
 
-const char*
+char*
 webgit_mimetype (const char* name);
 
 #endif