From: Julien Moutte <julien@moutte.net>
Date: Tue, 27 Jul 2004 10:10:02 +0000 (+0000)
Subject: sys/xvimage/xvimagesink.c: Some fixes to image size calculation.
X-Git-Tag: BRANCH-GSTREAMER-0_8-ROOT~796
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=23f23004c50bf9018b784baced9d5642f8755061;p=platform%2Fupstream%2Fgst-plugins-base.git

sys/xvimage/xvimagesink.c: Some fixes to image size calculation.

Original commit message from CVS:
2004-07-27  Julien MOUTTE  <julien@moutte.net>

* sys/xvimage/xvimagesink.c: (gst_xvimagesink_check_xshm_calls),
(gst_xvimagesink_xvimage_new): Some fixes to image size calculation.
---

diff --git a/ChangeLog b/ChangeLog
index d169d9c..6c29c1c 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2004-07-27  Julien MOUTTE  <julien@moutte.net>
+
+	* sys/xvimage/xvimagesink.c: (gst_xvimagesink_check_xshm_calls),
+	(gst_xvimagesink_xvimage_new): Some fixes to image size calculation.
+
 2004-07-27  Wim Taymans  <wim@fluendo.com>
 
 	* ext/libpng/gstpngdec.c: (gst_pngdec_src_getcaps):
diff --git a/sys/xvimage/xvimagesink.c b/sys/xvimage/xvimagesink.c
index 82183c4..a44cfa9 100644
--- a/sys/xvimage/xvimagesink.c
+++ b/sys/xvimage/xvimagesink.c
@@ -125,14 +125,13 @@ gst_xvimagesink_check_xshm_calls (GstXContext * xcontext)
   error_caught = FALSE;
   handler = XSetErrorHandler (gst_xvimagesink_handle_xerror);
 
-  xvimage->size = (xcontext->bpp / 8);
-
   /* Trying to create a 1x1 picture */
   xvimage->xvimage = XvShmCreateImage (xcontext->disp, xcontext->xv_port_id,
       xcontext->im_format, NULL, 1, 1, &xvimage->SHMInfo);
   if (!xvimage->xvimage)
     goto out;
 
+  xvimage->size = xvimage->xvimage->bytes_per_line;
   xvimage->SHMInfo.shmid = shmget (IPC_PRIVATE, xvimage->size,
       IPC_CREAT | 0777);
   xvimage->SHMInfo.shmaddr = shmat (xvimage->SHMInfo.shmid, 0, 0);
@@ -184,11 +183,6 @@ gst_xvimagesink_xvimage_new (GstXvImageSink * xvimagesink,
 
   g_mutex_lock (xvimagesink->x_lock);
 
-  xvimage->size =
-      (xvimagesink->xcontext->bpp / 8) * xvimage->width * xvimage->height;
-  GST_DEBUG_OBJECT (xvimagesink, "GStreamer's image size is %d, stride %d",
-      xvimage->size, xvimage->size / xvimage->height);
-
 #ifdef HAVE_XSHM
   if (xvimagesink->xcontext->use_xshm) {
     xvimage->xvimage = XvShmCreateImage (xvimagesink->xcontext->disp,
@@ -222,7 +216,8 @@ gst_xvimagesink_xvimage_new (GstXvImageSink * xvimagesink,
         xvimage->im_format, NULL, xvimage->width, xvimage->height);
 
     /* Allocating memory for image's data */
-    xvimage->xvimage->data = g_malloc (xvimage->xvimage->data_size);
+    xvimage->size = xvimage->xvimage->data_size;
+    xvimage->xvimage->data = g_malloc (xvimage->size);
 
     XSync (xvimagesink->xcontext->disp, FALSE);
   }