From 854630fb92f259a49d74d4b8c580ca479f402bae Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tim-Philipp=20M=C3=BCller?= Date: Wed, 7 Jan 2015 16:40:25 +0000 Subject: [PATCH] Revert "pngparse: improve parsing of the image" This reverts commit 59e7f0597d1cc524174e847de7b91407922304c8. This patch should not have been pushed, see https://bugzilla.gnome.org/show_bug.cgi?id=740058#c3 --- gst/videoparsers/gstpngparse.c | 9 +-------- gst/videoparsers/gstpngparse.h | 1 - 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/gst/videoparsers/gstpngparse.c b/gst/videoparsers/gstpngparse.c index 86fd9bcdb9..c4dbe3cf54 100644 --- a/gst/videoparsers/gstpngparse.c +++ b/gst/videoparsers/gstpngparse.c @@ -130,7 +130,7 @@ gst_png_parse_handle_frame (GstBaseParse * parse, GstByteReader reader; GstFlowReturn ret = GST_FLOW_OK; guint64 signature; - static guint width = 0, height = 0; + guint width = 0, height = 0; gst_buffer_map (frame->buffer, &map, GST_MAP_READ); gst_byte_reader_init (&reader, map.data, map.size); @@ -165,8 +165,6 @@ gst_png_parse_handle_frame (GstBaseParse * parse, } gst_byte_reader_skip (&reader, 8); - /* Skipping the data read in previous iterations */ - gst_byte_reader_skip (&reader, pngparse->skip_length); for (;;) { guint32 length; @@ -193,8 +191,6 @@ gst_png_parse_handle_frame (GstBaseParse * parse, if (!gst_byte_reader_skip (&reader, length + 4)) goto beach; - /* We have already read the data till now. Hence skip the read data next time we enter handle_frame() */ - pngparse->skip_length = gst_byte_reader_get_pos (&reader) + length + 4; if (code == GST_MAKE_FOURCC ('I', 'E', 'N', 'D')) { /* the start code and at least 2 empty frames (IHDR and IEND) */ @@ -238,9 +234,6 @@ gst_png_parse_handle_frame (GstBaseParse * parse, gst_buffer_unmap (frame->buffer, &map); - pngparse->skip_length = 0; - width = 0; - height = 0; return gst_base_parse_finish_frame (parse, frame, gst_byte_reader_get_pos (&reader)); } diff --git a/gst/videoparsers/gstpngparse.h b/gst/videoparsers/gstpngparse.h index 2142358b43..35744813c0 100644 --- a/gst/videoparsers/gstpngparse.h +++ b/gst/videoparsers/gstpngparse.h @@ -49,7 +49,6 @@ struct _GstPngParse guint width; guint height; - guint32 skip_length; gboolean sent_codec_tag; };