glib r7686 - in trunk/gio: . tests

[matthiasc svn gnome org]

Author: matthiasc
Date: Fri Nov 28 05:57:07 2008
New Revision: 7686
URL: http://svn.gnome.org/viewvc/glib?rev=7686&view=rev

Log:
2008-11-28  Matthias Clasen  <mclasen redhat com>

        Bug 562393 â g_buffered_input_stream_read_byte broken if data
        available

        * gio/gbufferedinputstream.c (g_buffered_input_stream_read_byte): Fix
        handling of buffered content.  Patch by Philip Withnall

        * gio/tests/buffered-input-stream.c: Add a testcase for this bug.
        * gio/tests/Makefile.am: And build it




Added:
   trunk/gio/tests/buffered-input-stream.c
Modified:
   trunk/gio/ChangeLog
   trunk/gio/gbufferedinputstream.c
   trunk/gio/tests/Makefile.am

Modified: trunk/gio/gbufferedinputstream.c
==============================================================================
--- trunk/gio/gbufferedinputstream.c	(original)
+++ trunk/gio/gbufferedinputstream.c	Fri Nov 28 05:57:07 2008
@@ -878,7 +878,7 @@
 
   available = priv->end - priv->pos;
 
-  if (available < 1)
+  if (available != 0)
     {
       g_input_stream_clear_pending (input_stream);
       return priv->buffer[priv->pos++];

Modified: trunk/gio/tests/Makefile.am
==============================================================================
--- trunk/gio/tests/Makefile.am	(original)
+++ trunk/gio/tests/Makefile.am	Fri Nov 28 05:57:07 2008
@@ -23,7 +23,8 @@
 	g-file-info 		\
 	data-input-stream 	\
 	data-output-stream 	\
-	g-icon
+	g-icon			\
+	buffered-input-stream
 
 if OS_UNIX
 TEST_PROGS += live-g-file unix-streams desktop-app-info
@@ -50,6 +51,9 @@
 g_icon_SOURCES	= g-icon.c
 g_icon_LDADD	= $(progs_ldadd)
 
+buffered_input_stream_SOURCES	= buffered-input-stream.c
+buffered_input_stream_LDADD	= $(progs_ldadd)
+
 live_g_file_SOURCES	  = live-g-file.c
 live_g_file_LDADD	  = $(progs_ldadd)
 

Added: trunk/gio/tests/buffered-input-stream.c
==============================================================================
--- (empty file)
+++ trunk/gio/tests/buffered-input-stream.c	Fri Nov 28 05:57:07 2008
@@ -0,0 +1,60 @@
+/* GLib testing framework examples and tests
+ * Copyright (C) 2008 Red Hat, Inc.
+ * Authors: Matthias Clasen <mclasen redhat com>
+ *
+ * This work is provided "as is"; redistribution and modification
+ * in whole or in part, in any medium, physical or electronic is
+ * permitted without restriction.
+ *
+ * This work 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.
+ *
+ * In no event shall the authors or contributors be liable for any
+ * direct, indirect, incidental, special, exemplary, or consequential
+ * damages (including, but not limited to, procurement of substitute
+ * goods or services; loss of use, data, or profits; or business
+ * interruption) however caused and on any theory of liability, whether
+ * in contract, strict liability, or tort (including negligence or
+ * otherwise) arising in any way out of the use of this software, even
+ * if advised of the possibility of such damage.
+ */
+
+#include <glib/glib.h>
+#include <gio/gio.h>
+#include <stdlib.h>
+#include <string.h>
+
+static void
+test_read_byte (void)
+{
+  GInputStream *base;
+  GInputStream *in;
+
+  g_test_bug ("562393");
+
+  base = g_memory_input_stream_new_from_data ("abcdefghijk", -1, NULL);
+  in = g_buffered_input_stream_new (base);
+
+  g_assert_cmpint (g_buffered_input_stream_read_byte (G_BUFFERED_INPUT_STREAM (in), NULL, NULL), ==, 'a');
+  g_assert_cmpint (g_buffered_input_stream_read_byte (G_BUFFERED_INPUT_STREAM (in), NULL, NULL), ==, 'b');
+  g_assert_cmpint (g_buffered_input_stream_read_byte (G_BUFFERED_INPUT_STREAM (in), NULL, NULL), ==, 'c');
+
+  g_assert_cmpint (g_input_stream_skip (in, 3, NULL, NULL), ==, 3);
+  
+  g_assert_cmpint (g_buffered_input_stream_read_byte (G_BUFFERED_INPUT_STREAM (in), NULL, NULL), ==, 'g');
+}
+
+
+int
+main (int   argc,
+      char *argv[])
+{
+  g_type_init ();
+  g_test_init (&argc, &argv, NULL);
+  g_test_bug_base ("http://bugzilla.gnome.org/";);
+
+  g_test_add_func ("/buffered-input-stream/read-byte", test_read_byte);
+
+  return g_test_run();
+}