cleanup
[ppastats.git] / src / ppastats.c
index 754cf4e..c1c18d8 100644 (file)
@@ -1,20 +1,20 @@
 /*
* Copyright (C) 2011-2014 jeanfi@gmail.com
- *
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License as
* published by the Free Software Foundation; either version 2 of the
* License, or (at your option) any later version.
- *
* This program is distributed in the hope that it will be useful, but
* WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
* General Public License for more details.
- *
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
* 02110-1301 USA
+  Copyright (C) 2011-2014 jeanfi@gmail.com
+
+  This program is free software; you can redistribute it and/or
+  modify it under the terms of the GNU General Public License as
+  published by the Free Software Foundation; either version 2 of the
+  License, or (at your option) any later version.
+
+  This program is distributed in the hope that it will be useful, but
+  WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+  General Public License for more details.
+
+  You should have received a copy of the GNU General Public License
+  along with this program; if not, write to the Free Software
+  Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+  02110-1301 USA
  */
 
 #include <libintl.h>
 #include <stdlib.h>
 #include <string.h>
 
-#include "list.h"
-#include "lp_ws.h"
+#include <list.h>
+#include <lp_ws.h>
 #include <plog.h>
-#include "ppastats.h"
+#include <ppastats.h>
+
+static void arch_stats_free(struct arch_stats *arch)
+{
+       free(arch->name);
+       free(arch);
+}
+
+static struct distro_stats *distro_stats_new(const char *name)
+{
+       struct distro_stats *d;
+
+       d = malloc(sizeof(struct distro_stats));
+       d->name = strdup(name);
+       d->archs = NULL;
+       d->download_count = 0;
+       d->ddts = NULL;
+
+       return d;
+}
+
+static void distro_stats_free(struct distro_stats *distro)
+{
+       struct arch_stats **archs;
+
+       archs = distro->archs;
+       if (archs) {
+               while (*archs) {
+                       arch_stats_free(*archs);
+                       archs++;
+               }
+               free(distro->archs);
+       }
+
+       daily_download_total_list_free(distro->ddts);
+
+       free(distro->name);
+       free(distro);
+}
+
+static void distro_stats_list_free(struct distro_stats **distros)
+{
+       struct distro_stats **cur;
+
+       if (distros) {
+               cur = distros;
+               while (*cur) {
+                       distro_stats_free(*cur);
+                       cur++;
+               }
+               free(distros);
+       }
+}
+
+static void version_stats_free(struct version_stats *version)
+{
+       distro_stats_list_free(version->distros);
+       daily_download_total_list_free(version->daily_download_totals);
+
+       free(version->version);
+       free(version);
+}
+
+static void package_stats_free(struct package_stats *package)
+{
+       struct version_stats **versions;
+
+       versions = package->versions;
+       if (versions) {
+               while (*versions) {
+                       version_stats_free(*versions);
+                       versions++;
+               }
+               free(package->versions);
+       }
+       distro_stats_list_free(package->distros);
+       daily_download_total_list_free(package->daily_download_totals);
+       free(package->name);
+       free(package);
+}
+
+static struct package_stats *package_stats_new(const char *name)
+{
+       struct package_stats *p;
+
+       p = malloc(sizeof(struct package_stats));
+       p->name = strdup(name);
+       p->versions = NULL;
+       p->download_count = 0;
+       p->daily_download_totals = NULL;
+       p->distros = NULL;
+
+       return p;
+}
 
 static struct package_stats *get_package_stats(struct ppa_stats *stats,
                                               const char *name)
@@ -45,15 +138,9 @@ static struct package_stats *get_package_stats(struct ppa_stats *stats,
                p_cur++;
        }
 
-       p = malloc(sizeof(struct package_stats));
-       p->name = strdup(name);
-       p->versions = NULL;
-       p->download_count = 0;
-       p->daily_download_totals = NULL;
-       p->distros = NULL;
+       p = package_stats_new(name);
 
-       tmp = (struct package_stats **)list_add
-               ((void **)stats->packages, p);
+       tmp = (struct package_stats **)list_add((void **)stats->packages, p);
        free(stats->packages);
        stats->packages = tmp;
 
@@ -89,25 +176,12 @@ static struct version_stats *get_version_stats(struct package_stats *package,
        return v;
 }
 
-static struct distro_stats *distro_stats_new(const char *name)
-{
-       struct distro_stats *d;
-
-       d = malloc(sizeof(struct distro_stats));
-       d->name = strdup(name);
-       d->archs = NULL;
-       d->download_count = 0;
-       d->ddts = NULL;
-
-       return d;
-}
-
 static struct distro_stats *get_distro_stats(struct version_stats *version,
                                             const char *name)
 {
-       struct distro_stats **cur = version->distros;
-       struct distro_stats *d;
-       struct distro_stats **tmp;
+       struct distro_stats **cur, *d, **tmp;
+
+       cur = version->distros;
 
        while (cur && *cur) {
                d = *cur;
@@ -120,9 +194,8 @@ static struct distro_stats *get_distro_stats(struct version_stats *version,
 
        d = distro_stats_new(name);
 
-
        tmp = (struct distro_stats **)list_add((void **)version->distros,
-                                                  d);
+                                              d);
        free(version->distros);
        version->distros = tmp;
 
@@ -132,10 +205,9 @@ static struct distro_stats *get_distro_stats(struct version_stats *version,
 static struct arch_stats *get_arch_stats(struct distro_stats *distro,
                                         const char *name)
 {
-       struct arch_stats **cur = distro->archs;
-       struct arch_stats *a;
-       struct arch_stats **tmp;
+       struct arch_stats **cur, *a, **tmp;
 
+       cur = distro->archs;
        while (cur && *cur) {
                a = *cur;
 
@@ -228,7 +300,8 @@ pkg_add_distro(struct package_stats *pkg,
                pkg_distro = distro_stats_new(distro_name);
                tmp = (struct distro_stats **)list_add((void **)pkg->distros,
                                                       (void *)pkg_distro);
-               free(pkg->distros);
+               if (pkg->distros != tmp)
+                       free(pkg->distros);
                pkg->distros = tmp;
        }
 
@@ -335,78 +408,6 @@ create_ppa_stats(const char *owner,
        return ppa;
 }
 
-static void arch_stats_free(struct arch_stats *arch)
-{
-       free(arch->name);
-       free(arch);
-}
-
-static void distro_stats_free(struct distro_stats *distro)
-{
-       struct arch_stats **archs;
-
-       archs = distro->archs;
-       if (archs) {
-               while (*archs) {
-                       arch_stats_free(*archs);
-                       archs++;
-               }
-               free(distro->archs);
-       }
-
-       daily_download_total_list_free(distro->ddts);
-
-       free(distro->name);
-       free(distro);
-}
-
-static void distro_stats_list_free(struct distro_stats **distros)
-{
-       if (distros) {
-               while (*distros) {
-                       distro_stats_free(*distros);
-                       distros++;
-               }
-       }
-}
-
-static void version_stats_free(struct version_stats *version)
-{
-       struct distro_stats **distros;
-
-       distros = version->distros;
-       if (distros) {
-               while (*distros) {
-                       distro_stats_free(*distros);
-                       distros++;
-               }
-               free(version->distros);
-       }
-
-       daily_download_total_list_free(version->daily_download_totals);
-
-       free(version->version);
-       free(version);
-}
-
-static void package_stats_free(struct package_stats *package)
-{
-       struct version_stats **versions;
-
-       versions = package->versions;
-       if (versions) {
-               while (*versions) {
-                       version_stats_free(*versions);
-                       versions++;
-               }
-               free(package->versions);
-       }
-       distro_stats_list_free(package->distros);
-       daily_download_total_list_free(package->daily_download_totals);
-       free(package->name);
-       free(package);
-}
-
 void ppa_stats_free(struct ppa_stats *ppastats)
 {
        struct package_stats **packages;