Bug#773451: libqtcore4:amd64: QRectF.setTop() and co do not play nicely with non-numbers (NaN and INF)

[Lisandro Damián Nicanor Pérez Meyer]

tag 773451 wontfix
thanks

On Thursday 18 December 2014 15:27:39 Vincent Fourmond wrote:
 Package: libqtcore4
 Version: 4:4.8.6+git64-g5dc8b2b+dfsg-2+b1
 Severity: normal
[snip]
   However, this is very confusing at best... Looking at the
   documentation, calling moveTop and then setBottom may be smarter,
   but, it really should just work...

Qt4 is on maintainance mode. This kind of changes are not allowed in upstream 
anymore, so even if considered a bug it won't be fixed.

I suggest you to take a look at Qt5 and see if you can see the same behaviour 
there. If you happen to be able to reproduce it the best place to fill a bug 
is directly upstream, feel free to CC me in the bug entry if you like.

Kinds regards, Lisandro.


-- 
Programming today is a race between software engineers striving to build
bigger and better idiot-proof programs, and the Universe trying to produce
bigger and better idiots. So far, the Universe is winning.
  Anonymous

Lisandro Damián Nicanor Pérez Meyer
http://perezmeyer.com.ar/
http://perezmeyer.blogspot.com/


signature.asc
Description: This is a digitally signed message part.

Bug#773451: libqtcore4:amd64: QRectF.setTop() and co do not play nicely with non-numbers (NaN and INF)

[Vincent Fourmond]

Package: libqtcore4
Version: 4:4.8.6+git64-g5dc8b2b+dfsg-2+b1
Severity: normal

  Hello,

  I've been hit by a bug where I have essentially this: (ret is QRectF)

ret.setTop(0.1);
ret.setBottom(-0.1);
o  Fxd:   ret.height()   -   ret.top()   | 
   ret.bottom()  endl;

  The output line shows that, if you have a NaN in the coordinates,
  then the line above DOES NOT set the top and bottom coordinates
  (they stay at NaN):

Fxd: nan - nan | nan

  Looking at the code for setTop, it all makes sense, as the pro

inline void QRectF::setTop(qreal pos) { qreal diff = pos - yp; yp +=
diff; h -= diff; }

  However, this is very confusing at best... Looking at the
  documentation, calling moveTop and then setBottom may be smarter,
  but, it really should just work...

  Best regards,

   Vincent

  
  



-- System Information:
Debian Release: 8.0
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing'), (500, 'stable'), (1, 
'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.14-2-amd64 (SMP w/8 CPU cores)
Locale: LANG=en_GB.utf8, LC_CTYPE=en_GB.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages libqtcore4:amd64 depends on:
ii  libc6  2.19-13
ii  libgcc11:4.9.2-8
ii  libglib2.0-0   2.42.1-1
ii  libstdc++6 4.9.2-8
ii  multiarch-support  2.19-13
ii  qtcore4-l10n   4:4.8.6+git64-g5dc8b2b+dfsg-2
ii  zlib1g 1:1.2.8.dfsg-2+b1

libqtcore4:amd64 recommends no packages.

Versions of packages libqtcore4:amd64 suggests:
ii  libicu52  52.1-6
ii  libthai0  0.1.21-1

-- no debconf information


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org