[gstreamer-bugs] [Bug 636049] ximagesrc: fix remote X and off by ones
GStreamer (bugzilla.gnome.org)
bugzilla at gnome.org
Sat Dec 18 06:33:23 PST 2010
https://bugzilla.gnome.org/show_bug.cgi?id=636049
GStreamer | gst-plugins-good | git
--- Comment #6 from Sebastian Dröge <slomo at circular-chaos.org> 2010-12-18 14:33:21 UTC ---
(In reply to comment #3)
> > > if (rects[i].x + rects[i].width - 1 < ximagesrc->startx
> >
> > So, if we have a rect x=0,width=2 and startx=2, there would be no overlap. The
> > rect would be on the pixel 0 and the pixel 1 but not on the pixel 2, where the
> > ximagesrc window starts. Anything I'm missing here?
>
> That's right, and the test will yield (0+2-1 < 2), which is true, so the code
> earlies out. Unless I'm missing your point, the case you mention seems to do as
> it should.
Hm, yes. You're right. I must've been confused, my comment explained why your
patch is correct :)
I'll push your changes now
--
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.
More information about the Gstreamer-bugs
mailing list