Fix compiler warnings
authorTom Hughes <tom@compton.nu>
Thu, 14 Feb 2013 11:52:27 +0000 (11:52 +0000)
committerTom Hughes <tom@compton.nu>
Thu, 14 Feb 2013 11:52:27 +0000 (11:52 +0000)
output_osm.c
planet06_pg.cpp
users.cpp

index 94dc9eb..180d7c5 100644 (file)
@@ -1,5 +1,6 @@
 #include <string.h>
 #include <assert.h>
 #include <string.h>
 #include <assert.h>
+#include <ctype.h>
 #include <inttypes.h>
 #include <stdint.h>
 #include <stdio.h>
 #include <inttypes.h>
 #include <stdint.h>
 #include <stdio.h>
@@ -135,7 +136,7 @@ void osm_node(int64_t id, long double lat, long double lon, struct keyval *tags,
     osm_tags(tags);
     printf(INDENT "</node>\n");
   } else {
     osm_tags(tags);
     printf(INDENT "</node>\n");
   } else {
-    printf(INDENT "<node id=\"%d\" lat=\"%.7Lf\" lon=\"%.7Lf\" "
+    printf(INDENT "<node id=\"%"PRId64"\" lat=\"%.7Lf\" lon=\"%.7Lf\" "
           "timestamp=\"%s\" version=\"%d\" changeset=\"%d\"%s/>\n", 
           id, lat, lon, ts, version, changeset, user);
   }
           "timestamp=\"%s\" version=\"%d\" changeset=\"%d\"%s/>\n", 
           id, lat, lon, ts, version, changeset, user);
   }
@@ -154,7 +155,7 @@ void osm_way(int64_t id, struct keyval *nodes, struct keyval *tags, const char *
     osm_tags(tags);
     printf(INDENT "</way>\n");
   } else {
     osm_tags(tags);
     printf(INDENT "</way>\n");
   } else {
-    printf(INDENT "<way id=\"%d\" timestamp=\"%s\" version=\"%d\" changeset=\"%d\"%s/>\n", id, ts, version, changeset, user);
+    printf(INDENT "<way id=\"%"PRId64"\" timestamp=\"%s\" version=\"%d\" changeset=\"%d\"%s/>\n", id, ts, version, changeset, user);
   }
 }
 
   }
 }
 
@@ -177,7 +178,7 @@ void osm_relation(int64_t id, struct keyval *members, struct keyval *roles, stru
     osm_tags(tags);
     printf(INDENT "</relation>\n");
   } else {
     osm_tags(tags);
     printf(INDENT "</relation>\n");
   } else {
-    printf(INDENT "<relation id=\"%d\" timestamp=\"%s\" version=\"%d\" changeset=\"%d\"%s/>\n", id, ts, version, changeset, user);
+    printf(INDENT "<relation id=\"%"PRId64"\" timestamp=\"%s\" version=\"%d\" changeset=\"%d\"%s/>\n", id, ts, version, changeset, user);
   }
 }
 
   }
 }
 
index 775c63b..4239f8a 100644 (file)
@@ -42,7 +42,6 @@ const char *reformDate(const char *str)
 {
     static char out[64], prev[64]; // Not thread safe
 
 {
     static char out[64], prev[64]; // Not thread safe
 
-    time_t tmp;
     struct tm tm;
 
     // Re-use the previous answer if we asked to convert the same timestamp twice
     struct tm tm;
 
     // Re-use the previous answer if we asked to convert the same timestamp twice
@@ -246,7 +245,6 @@ void changesets(pqxx::work &xaction) {
         itr != res.end(); ++itr) {
       const pqxx::result::tuple &row = *itr;
       const int64_t id = row[0].as<int64_t>();
         itr != res.end(); ++itr) {
       const pqxx::result::tuple &row = *itr;
       const int64_t id = row[0].as<int64_t>();
-      const int num_changes = row[4].as<int>();
       const bool null_bbox = row[5].is_null() || row[6].is_null() || row[7].is_null() || row[8].is_null();
 
       if (!tagstream.get(id, &tags)) {
       const bool null_bbox = row[5].is_null() || row[6].is_null() || row[7].is_null() || row[8].is_null();
 
       if (!tagstream.get(id, &tags)) {
@@ -260,7 +258,7 @@ void changesets(pqxx::work &xaction) {
                    lookup_user(row[1].c_str()), // user_id
                    created_at, // created_at
                    reformDate(row[3].c_str()), // closed_at
                    lookup_user(row[1].c_str()), // user_id
                    created_at, // created_at
                    reformDate(row[3].c_str()), // closed_at
-            row[4].as<int>(), // num_changes
+                   row[4].as<int>(), // num_changes
                    null_bbox ? 0 : 1,
                    null_bbox ? 0 : row[5].as<int>() / SCALE, // min_lat
                    null_bbox ? 0 : row[6].as<int>() / SCALE, // max_lat
                    null_bbox ? 0 : 1,
                    null_bbox ? 0 : row[5].as<int>() / SCALE, // min_lat
                    null_bbox ? 0 : row[6].as<int>() / SCALE, // max_lat
index dea1ce4..8b452d7 100644 (file)
--- a/users.cpp
+++ b/users.cpp
@@ -78,7 +78,7 @@ void fetch_users(pqxx::work &xaction)
 }
 
 void free_users(void) {
 }
 
 void free_users(void) {
-  for(int i=0; i<=max_uid; i++)
+  for(unsigned long i=0; i<=max_uid; i++)
     free(user_list[i]);
   free(user_list);
 }
     free(user_list[i]);
   free(user_list);
 }