codecparsers: vc1: simplify GstVC1VopDquant structure.

Rename dqsbedge to dqbedge. The intent is that we can only have a single
boundary edge selector, depending on the value of dqprofile. So, dqbedge
represents DQSBEDGE if dqprofile == GST_VC1_DQPROFILE_SINGLE_EDGE, or
DQDBEDGE if dqprofile == GST_VC1_DQPROFILE_DOUBLE_EDGE.

The former dqbedge field is marked as unused and can be removed on the
next gst-plugins-bad version that allows ABI changes.

https://bugzilla.gnome.org/show_bug.cgi?id=692272

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>
This commit is contained in:
Gwenole Beauchesne 2013-01-22 11:42:49 +01:00
parent 4202eeff7d
commit cbec9bfcd0
2 changed files with 7 additions and 7 deletions

View file

@ -646,7 +646,7 @@ parse_vopdquant (GstBitReader * br, GstVC1FrameHdr * framehdr, guint8 dquant)
switch (vopdquant->dqprofile) {
case GST_VC1_DQPROFILE_SINGLE_EDGE:
case GST_VC1_DQPROFILE_DOUBLE_EDGES:
READ_UINT8 (br, vopdquant->dqsbedge, 2);
READ_UINT8 (br, vopdquant->dqbedge, 2);
break;
case GST_VC1_DQPROFILE_ALL_MBS:

View file

@ -485,14 +485,14 @@ struct _GstVC1VopDquant
guint8 dquantfrm;
guint8 dqprofile;
/* if dqprofile == GST_VC1_DQPROFILE_SINGLE_EDGE
* or GST_VC1_DQPROFILE_DOUBLE_EDGE:*/
guint8 dqsbedge;
/* if dqprofile == GST_VC1_DQPROFILE_SINGLE_EDGE
* or GST_VC1_DQPROFILE_DOUBLE_EDGE:*/
/* Boundary edge selection. This represents DQSBEDGE
* if dqprofile == GST_VC1_DQPROFILE_SINGLE_EDGE or
* DQDBEDGE if dqprofile == GST_VC1_DQPROFILE_DOUBLE_EDGE */
guint8 dqbedge;
/* FIXME: remove */
guint8 unused;
/* if dqprofile == GST_VC1_DQPROFILE_ALL_MBS */
guint8 dqbilevel;