----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/378/#review606 -----------------------------------------------------------
Ship it! looks fine to me. did you do any testing? you left the 'Testing done' section blank. Do you have a SVN account that you can use to commit this, or do I need to commit it? - Matt On 2009-03-21 10:46:58, Brian Johnson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/378/ > ----------------------------------------------------------- > > (Updated 2009-03-21 10:46:58) > > > Review request for Kopete. > > > Summary > ------- > > If one of the v4l2 controls has a minimum value other then 0 kopete will not > correctly calculate value to set the control to. For instance in my webcam > driver the hue is done using a range of > -180 to +180, however when changing the hue using kopete it will only > calculate values between 0 and 360. This patch should fix the issue. > > > Diffs > ----- > > trunk/KDE/kdenetwork/kopete/libkopete/avdevice/videodevice.cpp 942363 > > Diff: http://reviewboard.kde.org/r/378/diff > > > Testing > ------- > > > Thanks, > > Brian > > _______________________________________________ kopete-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kopete-devel
