Add -Wall to compiler flags and remove all remaining warnings

src/Makefile.am

@@ -1,9 +1,10 @@
 MAINTAINERCLEANFILES = Makefile.in
 
-INCLUDES = -I. \
-           -I$(top_srcdir) \
-           -I$(includedir) \
-           @E_CFLAGS@
+AM_CPPFLAGS = -Wall \
+              -I. \
+              -I$(top_srcdir) \
+              -I$(includedir) \
+              @E_CFLAGS@
 
 pkgdir = $(datadir)/$(MODULE_ARCH)
 pkg_LTLIBRARIES = module.la

src/e_mod_main.c

@@ -42,7 +42,7 @@ e_modapi_shutdown(E_Module *m)
    e_configure_registry_item_del("appearance/penguins");
    e_configure_registry_category_del("appearance");
 
-   if (pop = m->data)
+   if ((pop = m->data))
    {
       E_FREE_FUNC(pop->config_dialog, e_object_del);
       penguins_shutdown(pop);

src/e_mod_penguins.c

@@ -60,7 +60,6 @@ Penguins_Population *
 penguins_init(E_Module *m)
 {
    Penguins_Population *pop;
-   Eina_List *managers, *l, *l2;
    char buf[PATH_MAX];
 
    pop = E_NEW(Penguins_Population, 1);
@@ -106,10 +105,11 @@ penguins_init(E_Module *m)
       if (eina_str_has_suffix(filename, ".edj"))
       {
          snprintf(buf, sizeof(buf), "%s/themes/%s", e_module_dir_get(m), filename);
-         if (name = edje_file_data_get(buf, "PopulationName"))
+         if ((name = edje_file_data_get(buf, "PopulationName")))
          {
             printf("PENGUINS: Theme found: %s (%s)\n", filename, name);
             pop->themes = eina_list_append(pop->themes, strdup(buf));
+            free(name);
          }
       }
       free(filename);