]> git.street.me.uk Git - andy/viking.git/commitdiff
projects / andy / viking.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: ae941b4)
Refactoring: merge a_check_html_file and a_check_kml_file
authorGuilhem Bonnefille <guilhem.bonnefille@gmail.com>
Fri, 18 Sep 2009 21:13:44 +0000 (23:13 +0200)
committerGuilhem Bonnefille <guilhem.bonnefille@gmail.com>
Fri, 18 Sep 2009 21:13:44 +0000 (23:13 +0200)
src/download.c patch | blob | blame | history

diff --git a/src/download.c b/src/download.c
index 9aefb479b31ef802835c11890c19c0f2ba84c1b4..1f10ed124a06dddd0217f70abf21b6b59a55ecfd 100644 (file)
--- a/src/download.c
+++ b/src/download.c
@@ -34,20 +34,13 @@
 
 #include "curl_download.h"
 
-gboolean a_check_html_file(FILE* f)
+static gboolean check_file_first_line(FILE* f, gchar *patterns[])
 {
   gchar **s;
   gchar *bp;
   fpos_t pos;
   gchar buf[33];
   size_t nr;
-  gchar * html_str[] = {
-    "<html",
-    "<!DOCTYPE html",
-    "<head",
-    "<title",
-    NULL
-  };
 
   memset(buf, 0, sizeof(buf));
   fgetpos(f, &pos);
@@ -60,46 +53,40 @@ gboolean a_check_html_file(FILE* f)
   }
   if ((bp >= (buf + sizeof(buf) -1)) || ((bp - buf) >= nr))
     return FALSE;
-  for (s = html_str; *s; s++) {
+  for (s = patterns; *s; s++) {
     if (strncasecmp(*s, bp, strlen(*s)) == 0)
       return TRUE;
   }
   return FALSE;
 }
 
+gboolean a_check_html_file(FILE* f)
+{
+  gchar * html_str[] = {
+    "<html",
+    "<!DOCTYPE html",
+    "<head",
+    "<title",
+    NULL
+  };
+
+  return check_file_first_line(f, html_str);
+}
+
 gboolean a_check_map_file(FILE* f)
 {
+  /* FIXME no more true since a_check_kml_file */
   return !a_check_html_file(f);
 }
 
 gboolean a_check_kml_file(FILE* f)
 {
-  gchar **s;
-  gchar *bp;
-  fpos_t pos;
-  gchar buf[33];
-  size_t nr;
-  gchar * html_str[] = {
+  gchar * kml_str[] = {
     "<?xml",
     NULL
   };
 
-  memset(buf, 0, sizeof(buf));
-  fgetpos(f, &pos);
-  rewind(f);
-  nr = fread(buf, 1, sizeof(buf) - 1, f);
-  fsetpos(f, &pos);
-  for (bp = buf; (bp < (buf + sizeof(buf) - 1)) && (nr > (bp - buf)); bp++) {
-    if (!(isspace(*bp)))
-      break;
-  }
-  if ((bp >= (buf + sizeof(buf) -1)) || ((bp - buf) >= nr))
-    return FALSE;
-  for (s = html_str; *s; s++) {
-    if (strncasecmp(*s, bp, strlen(*s)) == 0)
-      return TRUE;
-  }
-  return FALSE;
+  return check_file_first_line(f, kml_str);
 }
 
 static int download( const char *hostname, const char *uri, const char *fn, DownloadOptions *options, gboolean ftp)
Viking GPS editor