Skip to content

Fix conversion of large integers to quint64 #263

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
usiems merged 1 commit into from
May 28, 2025
Merged

Fix conversion of large integers to quint64 #263

usiems merged 1 commit into from
May 28, 2025

Conversation

@usiems
Copy link
Contributor

@usiems usiems commented May 23, 2025 •
edited
Loading

Values in the range of 0x7FFFFFFFFFFFFFFF to 0xFFFFFFFFFFFFFFFF caused an OverflowError when converted to quint64.

How to reproduce:

from PythonQt.QtGui import QPalette
p = QPalette()
# QPalette.setResolveMask takes a quint64 argument, so this should work:
p.setResolveMask(0x8FFFFFFFFFFFFFFF)

@usiems
Fix conversion of large integers to quint64
e2874d1 
Values in the range of 0x7FFFFFFFFFFFFFFF to 0xFFFFFFFFFFFFFFFF caused
an OverflowError when converted to quint64
@usiems usiems force-pushed the fix_quint64_conversion branch from f5c30de to e2874d1 Compare May 28, 2025 11:36
@usiems usiems merged commit efb3f7c into master May 28, 2025
19 checks passed
@usiems usiems deleted the fix_quint64_conversion branch May 28, 2025 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mrbean-bremen mrbean-bremen mrbean-bremen approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@usiems @mrbean-bremen