sys/xvimage/xvimagesink.c: Don't set colour balance values on the Xv port if the user hasn't changed them (via proper...

Original commit message from CVS:
* sys/xvimage/xvimagesink.c:
Don't set colour balance values on the Xv port if the user hasn't
changed them (via properties or the interface). Avoids accumulating
rounding errors for the common case.
Partial fix for bug #537889.
This commit is contained in:
Michael Smith 2008-06-20 17:02:48 +00:00
parent 9d2564874a
commit 23f5a075ab
2 changed files with 13 additions and 0 deletions

View file

@ -1,3 +1,11 @@
2008-06-20 Michael Smith <msmith@songbirdnest.com>
* sys/xvimage/xvimagesink.c:
Don't set colour balance values on the Xv port if the user hasn't
changed them (via properties or the interface). Avoids accumulating
rounding errors for the common case.
Partial fix for bug #537889.
2008-06-20 Michael Smith <msmith@songbirdnest.com>
* gst/playback/gstdecodebin2.c:

View file

@ -1027,6 +1027,11 @@ gst_xvimagesink_update_colorbalance (GstXvImageSink * xvimagesink)
if (xvimagesink->xcontext == NULL)
return;
/* Don't set the attributes if they haven't been changed, to avoid
* rounding errors changing the values */
if (!xvimagesink->cb_changed)
return;
/* For each channel of the colorbalance we calculate the correct value
doing range conversion and then set the Xv port attribute to match our
values. */