added module to handle log

[ppastats.git] / src / ppastats.c

diff --git a/src/ppastats.c b/src/ppastats.c
index fdcd6e1..7861301 100644 (file)
--- a/src/ppastats.c
+++ b/src/ppastats.c
@@ -17,11 +17,15 @@
     02110-1301 USA
 */
 
+#include <libintl.h>
+#define _(String) gettext(String)
+
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 
 #include "list.h"
+#include "log.h"
 #include "lp_ws.h"
 #include "ppastats.h"
 
@@ -247,7 +251,7 @@ create_ppa_stats(const char *owner,
        free(ppa_url);
 
        if (!history) {
-               fprintf(stderr, "Failed to retrieve PPA information\n");
+               log_err(_("Failed to retrieve PPA information"));
                exit(EXIT_FAILURE);
        }
 
@@ -258,11 +262,20 @@ create_ppa_stats(const char *owner,
        ppa->daily_download_totals = NULL;
        ppa->download_count = 0;
 
-       h_cur = history;
-       while (*h_cur) {
+       for (h_cur = history; *h_cur; ++h_cur) {
                h = *h_cur;
                totals = get_daily_download_totals(h->self_link);
+               if (!totals) {
+                       log_err(_("Failed to retrieve download totals for %s"),
+                               h->self_link);
+                       continue;
+               }
                count = get_download_count(h->self_link);
+               if (count == -1) {
+                       log_err(_("Failed to retrieve download count for %s"),
+                               h->self_link);
+                       count = 0;
+               }
                pkg_name = h->binary_package_name;
                pkg_version = h->binary_package_version;
                arch_series
@@ -293,8 +306,6 @@ create_ppa_stats(const char *owner,
                pkg_add_distro(pkg, distro_series->name, count, totals);
 
                daily_download_total_list_free(totals);
-
-               h_cur++;
        }
 
        binary_package_publishing_history_list_free(history);