Trim trailing white spaces on empty lines.

[geeqie.git] / src / histogram.c

diff --git a/src/histogram.c b/src/histogram.c
index 8f65376..c11b689 100644 (file)
--- a/src/histogram.c
+++ b/src/histogram.c
@@ -1,6 +1,6 @@
 /*
  * Geeqie
- * Copyright (C) 2008 - 2009 The Geeqie Team
+ * Copyright (C) 2008 - 2012 The Geeqie Team
  *
  * Author: Vladimir Nadvornik
  * based on a patch by Uwe Ohse
@@ -31,8 +31,8 @@ struct _HistMap {
        gulong g[HISTMAP_SIZE];
        gulong b[HISTMAP_SIZE];
        gulong max[HISTMAP_SIZE];
-       
-       gint idle_id;
+
+       guint idle_id; /* event source id */
        GdkPixbuf *pixbuf;
        gint y;
 };
@@ -43,7 +43,7 @@ Histogram *histogram_new(void)
        Histogram *histogram;
 
        histogram = g_new0(Histogram, 1);
-       histogram->histogram_channel = HCHAN_RGB;
+       histogram->histogram_channel = HCHAN_DEFAULT;
        histogram->histogram_mode = 0;
 
        /* grid */
@@ -104,7 +104,7 @@ gint histogram_toggle_mode(Histogram *histogram)
 const gchar *histogram_label(Histogram *histogram)
 {
        const gchar *t1 = "";
-       
+
        if (!histogram) return NULL;
 
        if (histogram->histogram_mode)
@@ -131,14 +131,13 @@ const gchar *histogram_label(Histogram *histogram)
 static HistMap *histmap_new(void)
 {
        HistMap *histmap = g_new0(HistMap, 1);
-       histmap->idle_id = -1;
        return histmap;
 }
 
 void histmap_free(HistMap *histmap)
 {
        if (!histmap) return;
-       if (histmap->idle_id != -1) g_source_remove(histmap->idle_id);
+       if (histmap->idle_id) g_source_remove(histmap->idle_id);
        if (histmap->pixbuf) g_object_unref(histmap->pixbuf);
        g_free(histmap);
 }
@@ -148,13 +147,13 @@ static gboolean histmap_read(HistMap *histmap, gboolean whole)
        gint w, h, i, j, srs, has_alpha, step, end_line;
        guchar *s_pix;
        GdkPixbuf *imgpixbuf = histmap->pixbuf;
-       
+
        w = gdk_pixbuf_get_width(imgpixbuf);
        h = gdk_pixbuf_get_height(imgpixbuf);
        srs = gdk_pixbuf_get_rowstride(imgpixbuf);
        s_pix = gdk_pixbuf_get_pixels(imgpixbuf);
        has_alpha = gdk_pixbuf_get_has_alpha(imgpixbuf);
-       
+
        if (whole)
                {
                end_line = h;
@@ -175,7 +174,7 @@ static gboolean histmap_read(HistMap *histmap, gboolean whole)
                        guint max = sp[0];
                        if (sp[1] > max) max = sp[1];
                        if (sp[2] > max) max = sp[2];
-               
+
                        histmap->r[sp[0]]++;
                        histmap->g[sp[1]]++;
                        histmap->b[sp[2]]++;
@@ -185,13 +184,13 @@ static gboolean histmap_read(HistMap *histmap, gboolean whole)
                        }
                }
        histmap->y = end_line;
-       return end_line >= h;   
+       return end_line >= h;
 }
 
 const HistMap *histmap_get(FileData *fd)
 {
-       if (fd->histmap && fd->histmap->idle_id == -1) return fd->histmap; /* histmap exists and is finished */
-       
+       if (fd->histmap && !fd->histmap->idle_id) return fd->histmap; /* histmap exists and is finished */
+
        return NULL;
 }
 
@@ -203,7 +202,7 @@ static gboolean histmap_idle_cb(gpointer data)
                /* finished */
                g_object_unref(fd->histmap->pixbuf); /*pixbuf is no longer needed */
                fd->histmap->pixbuf = NULL;
-               fd->histmap->idle_id = -1;
+               fd->histmap->idle_id = 0;
                file_data_send_notification(fd, NOTIFY_HISTMAP);
                return FALSE;
                }
@@ -227,7 +226,7 @@ static void histogram_vgrid(Histogram *histogram, GdkPixbuf *pixbuf, gint x, gin
 {
        guint i;
        float add;
-       
+
        if (histogram->vgrid == 0) return;
 
        add = width / (float)histogram->vgrid;
@@ -248,7 +247,7 @@ static void histogram_hgrid(Histogram *histogram, GdkPixbuf *pixbuf, gint x, gin
 {
        guint i;
        float add;
-       
+
        if (histogram->hgrid == 0) return;
 
        add = height / (float)histogram->hgrid;
@@ -256,7 +255,7 @@ static void histogram_hgrid(Histogram *histogram, GdkPixbuf *pixbuf, gint x, gin
        for (i = 1; i < histogram->hgrid; i++)
                {
                gint ypos = y + (int)(i * add + 0.5);
-       
+
                pixbuf_draw_line(pixbuf, x, y, width, height, x, ypos, x + width, ypos,
                                 histogram->grid_color.R,
                                 histogram->grid_color.G,
@@ -273,18 +272,20 @@ gboolean histogram_draw(Histogram *histogram, const HistMap *histmap, GdkPixbuf
        gdouble logmax;
        gint combine = (HISTMAP_SIZE - 1) / width + 1;
        gint ypos = y + height;
-       
+
        if (!histogram || !histmap) return FALSE;
-       
+
        /* Draw the grid */
        histogram_vgrid(histogram, pixbuf, x, y, width, height);
        histogram_hgrid(histogram, pixbuf, x, y, width, height);
 
-       for (i = 0; i < HISTMAP_SIZE; i++)
+       /* exclude overexposed and underexposed */
+       for (i = 1; i < HISTMAP_SIZE - 1; i++)
                {
                if (histmap->r[i] > max) max = histmap->r[i];
                if (histmap->g[i] > max) max = histmap->g[i];
                if (histmap->b[i] > max) max = histmap->b[i];
+               if (histmap->max[i] > max) max = histmap->max[i];
                }
 
        if (max > 0)
@@ -301,6 +302,7 @@ gboolean histogram_draw(Histogram *histogram, const HistMap *histmap, GdkPixbuf
                gint bplus = 0;
                gint ii = i * HISTMAP_SIZE / width;
                gint xpos = x + i;
+               gint num_chan;
 
                for (j = 0; j < combine; j++)
                        {
@@ -310,19 +312,25 @@ gboolean histogram_draw(Histogram *histogram, const HistMap *histmap, GdkPixbuf
                        v[2] += histmap->b[p];
                        v[3] += histmap->max[p];
                        }
-       
+
                for (j = 0; combine > 1 && j < 4; j++)
                        v[j] /= combine;
-               
-               for (j = 0; j < 4; j++)
+
+               num_chan = (histogram->histogram_channel == HCHAN_RGB) ? 3 : 1;
+               for (j = 0; j < num_chan; j++)
                        {
-                       gint chanmax = HCHAN_R;
-               
-                       if (v[HCHAN_G] > v[HCHAN_R]) chanmax = HCHAN_G;
-                       if (v[HCHAN_B] > v[HCHAN_G]) chanmax = HCHAN_B;
-                               
-                       if (histogram->histogram_channel >= HCHAN_RGB
-                           || chanmax == histogram->histogram_channel)
+                       gint chanmax;
+                       if (histogram->histogram_channel == HCHAN_RGB)
+                               {
+                               chanmax = HCHAN_R;
+                               if (v[HCHAN_G] > v[HCHAN_R]) chanmax = HCHAN_G;
+                               if (v[HCHAN_B] > v[chanmax]) chanmax = HCHAN_B;
+                               }
+                       else
+                               {
+                               chanmax = histogram->histogram_channel;
+                               }
+
                                {
                                gulong pt;
                                gint r = rplus;
@@ -349,7 +357,7 @@ gboolean histogram_draw(Histogram *histogram, const HistMap *histmap, GdkPixbuf
                                        case HCHAN_B:   r = 0;          g = 0;  break;
                                        case HCHAN_MAX: r = 0;  b = 0;  g = 0;  break;
                                        }
-                               
+
                                if (v[chanmax] == 0)
                                        pt = 0;
                                else if (histogram->histogram_mode)
@@ -372,8 +380,9 @@ gboolean histogram_draw(Histogram *histogram, const HistMap *histmap, GdkPixbuf
 
 void histogram_notify_cb(FileData *fd, NotifyType type, gpointer data)
 {
-       if ((type & (NOTIFY_CHANGE || NOTIFY_REREAD)) && fd->histmap)
+       if ((type & NOTIFY_REREAD) && fd->histmap)
                {
+               DEBUG_1("Notify histogram: %s %04x", fd->path, type);
                histmap_free(fd->histmap);
                fd->histmap = NULL;
                }