added a comment to explain this function better.

1999-12-03  Jonathan Blandford  <jrb@redhat.com>

	* gdk-pixbuf/io-gif.c (gif_fill_in_lines): added a comment to
	explain this function better.

diff --git a/gdk-pixbuf/ChangeLog b/gdk-pixbuf/ChangeLog
index 7881c0bc568bc93e7fc43a6ff81589d40a0852d8..2f51bd106481574f1a8422b16ca45fed293b123e 100644 (file)
--- a/gdk-pixbuf/ChangeLog
+++ b/gdk-pixbuf/ChangeLog
@@ -1,3 +1,8 @@
+1999-12-03  Jonathan Blandford  <jrb@redhat.com>
+
+       * gdk-pixbuf/io-gif.c (gif_fill_in_lines): added a comment to
+       explain this function better.
+
 1999-12-02  Federico Mena Quintero  <federico@redhat.com>
 
        Source files moved from src/ to gdk-pixbuf/, to be consistent with

diff --git a/gdk-pixbuf/io-gif.c b/gdk-pixbuf/io-gif.c
index b74a69a13457815f7ded38f00f781d43ab7101d9..e5cef27db377a8f7400c4517251d17757df4b318 100644 (file)
--- a/gdk-pixbuf/io-gif.c
+++ b/gdk-pixbuf/io-gif.c
@@ -628,11 +628,15 @@ gif_fill_in_lines (GifContext *context, guchar *dest, guchar v)
                        gif_fill_in_pixels (context, dest, 3, v);
                        gif_fill_in_pixels (context, dest, 4, v);
                }
+               /* we don't need a break here.  We draw the outer pixels first, then the
+                * inner ones, then the innermost ones.  case 0 needs to draw all 3 bands.
+                * case 1, just the last two, and case 2 just draws the last one*/
        case 1:
                if (context->draw_ypos > 2)
                        gif_fill_in_pixels (context, dest, -2, v);
                if (context->draw_ypos < (context->frame_height - 2))
                        gif_fill_in_pixels (context, dest, 2, v);
+               /* no break as above. */
        case 2:
                if (context->draw_ypos > 1)
                        gif_fill_in_pixels (context, dest, -1, v);