[Bug 741420] video pools: should update size in configuration after applying alignement

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=741420
  GStreamer | gst-plugins-base | 1.4.1

--- Comment #10 from kevin <kevinbing.song at gmail.com> 2014-12-16 06:45:20 UTC ---
  /* size should have been reset. This is not a catch all, pool with
   * size requirement per memory should do their own check. */
  if (gst_buffer_get_size (buffer) != pool->priv->size)
    goto discard;


  *buffer =
      gst_buffer_new_allocate (priv->allocator, info->size, &priv->params);



1. gst_buffer_get_size (buffer) will be info->size, not including alignment in
&priv->params.
2. pool->priv->size is set by gst_buffer_pool_config_set_params(). So it is
easy to keep it same.
3.  * @size: the size of each buffer, not including prefix and padding   the
comments on function: gst_buffer_pool_config_set_params(). is it mean memory
padding or video padding? it is mean memory alignment padding.

Is my understand right? What kind of use case don't know the acturally size
allocated? You mean don't know memory size or the size you set to
gst_buffer_new_allocate()?

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.