[Interest] Totally broken QRect::moveRight

Jason H jhihn at gmx.com
Tue Dec 9 21:13:13 CET 2014


I believe you are meaning to use a "translate" function. 

> Sent: Tuesday, December 09, 2014 at 3:16 AM
> From: "Igor Mironchik" <igor.mironchik at gmail.com>
> To: interest at qt-project.org
> Subject: Re: [Interest] Totally broken QRect::moveRight
>
> OMG, what happens when reading the documentation not carefully...
> 
> Thanks.
> 
> I expected that moveRight() moves rectangle to the given position, not  
> right edge.
> 
> On Tue, 09 Dec 2014 10:53:55 +0300, Thiago Macieira  
> <thiago.macieira at intel.com> wrote:
> 
> > On Tuesday 09 December 2014 09:45:13 Igor Mironchik wrote:
> >> Hi guys,
> >>
> >> I've found interesting bug in QRect in Qt 5.3.2.
> >>
> >> Look at this code:
> >>
> >> inline void QRect::moveRight(int pos)
> >> {
> >>      x1 += (pos - x2);
> >>      x2 = pos;
> >> }
> >>
> >>
> >> How it can work?
> >>
> >> QRect( 0, 0, 10, 10 ).moveRight( 10 )
> >>
> >> produces QRect( 1, 0, 10, 10 )...
> >
> > And as I replied in your bug report, there's nothing wrong with this.
> 
> 
> -- 
> Best Regards,
> Igor Mironchik.
> _______________________________________________
> Interest mailing list
> Interest at qt-project.org
> http://lists.qt-project.org/mailman/listinfo/interest
> 



More information about the Interest mailing list