After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 545566 - totem rounds brightness and contrast controls incorrectly
totem rounds brightness and contrast controls incorrectly
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: GStreamer backend
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Maintainer alias for GStreamer component of Totem
Maintainer alias for GStreamer component of Totem
Depends on: 537889
Blocks:
 
 
Reported: 2008-07-30 18:16 UTC by David Schleef
Modified: 2008-08-07 11:37 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description David Schleef 2008-07-30 18:16:51 UTC
+++ This bug was initially created as a clone of Bug #537889 +++

...

This still doesn't fix it entirely with totem, for two reasons:
 - totem sets the values every time. We could fix that by not scaling things (or by only scaling things for use via properties, not via the colourbalance interface), but...
 - totem reads these via the colorbalance interface, and does its own scaling, with rounding errors.

...

See patch on other bug.
Comment 1 Robin Stocker 2008-08-07 08:17:01 UTC
I can confirm the "colour drifting bug" and also that the patch in the other bug (together with gst-plugins-base from CVS) fixes it. Here are some other users having this problem:

https://bugs.launchpad.net/ubuntu/+source/totem/+bug/216820

Would be great if the patch was committed.
Comment 2 Tim-Philipp Müller 2008-08-07 11:37:07 UTC
 2008-08-07  Tim-Philipp Müller  <tim.muller at collabora co uk>

        Patch by: David Schleef <ds@schleef.org>

        * src/backend/bacon-video-widget-gst-0.10.c:
          (bacon_video_widget_get_video_property),
          (bacon_video_widget_set_video_property)::
          Fix rounding errors when converting colorbalance values to/from
          a different value range. Should fix drift in colour balance.
          (Closes: #545566, #537889)