Re: Frequent Crashes with Puddletag

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 26 May 2022 14:29:31 -0700
Samuel Sieb <samuel@xxxxxxxx> wrote:

> On 5/26/22 06:18, stan via users wrote:
> > So, the problem is in this function,
> > 
> >      def wheelEvent(self, e):
> >          h = self.horizontalScrollBar()
> >          if not self.verticalScrollBar().isVisible() and
> > h.isVisible(): numsteps = e.angleDelta().y() / 5  
> 
> Python3 changed the default for division.  Python2 would always give
> you an integer result from dividing integers, but Python3 will give
> you a float if the numbers are not evenly divisible.  The simplest
> fix would be to change the "/" to a "//" (integer division).

Good catch!  I attached to the argument 1 and got caught by the off by 1
error, the argument count starts at zero, not 1, and so was looking at
h.value() instead of numsteps.  
TypeError: setValue(self, int): argument 1 has unexpected type 'float'

> 
> >              h.setValue(h.value() - numsteps)       <--------  here
> >              e.accept()
> >          else:
> >              QTableView.wheelEvent(self, e)  
_______________________________________________
users mailing list -- users@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to users-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/users@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure



[Index of Archives]     [Older Fedora Users]     [Fedora Announce]     [Fedora Package Announce]     [EPEL Announce]     [EPEL Devel]     [Fedora Magazine]     [Fedora Summer Coding]     [Fedora Laptop]     [Fedora Cloud]     [Fedora Advisory Board]     [Fedora Education]     [Fedora Security]     [Fedora Scitech]     [Fedora Robotics]     [Fedora Infrastructure]     [Fedora Websites]     [Anaconda Devel]     [Fedora Devel Java]     [Fedora Desktop]     [Fedora Fonts]     [Fedora Marketing]     [Fedora Management Tools]     [Fedora Mentors]     [Fedora Package Review]     [Fedora R Devel]     [Fedora PHP Devel]     [Kickstart]     [Fedora Music]     [Fedora Packaging]     [Fedora SELinux]     [Fedora Legal]     [Fedora Kernel]     [Fedora OCaml]     [Coolkey]     [Virtualization Tools]     [ET Management Tools]     [Yum Users]     [Yosemite News]     [Gnome Users]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [Fedora Sparc]     [Libvirt Users]     [Fedora ARM]

  Powered by Linux