Skip to content
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

Remove debug output from QPainter #3780

Closed
magnesj opened this issue Nov 29, 2018 · 0 comments
Closed

Remove debug output from QPainter #3780

magnesj opened this issue Nov 29, 2018 · 0 comments
Assignees
@lindkvis
Labels
BugInRelease Bug in an official release
Milestone
Maintenance - Pat...

Comments

@magnesj
Copy link
Member

magnesj commented Nov 29, 2018 •
edited
Loading

QPainter::fontMetrics: Painter not active

Seen on Windows and Linux when summary curves with symbols are drawn.

Probably related to use of QPainter in
RiuQwtSymbol::labelBoundingRect
@magnesj magnesj added the BugInRelease Bug in an official release label Nov 29, 2018
@magnesj magnesj added this to the Maintenance Dec 2018 milestone Nov 29, 2018
lindkvis added a commit that referenced this issue Nov 30, 2018
@lindkvis
#3780 Get rid of debug output from painter
c60e6b8 
* don't really need the boundingRect override for RiuQwtSymbol
* And it needs a painter
* Just get rid of it and pass in the painter for the labelBoundingRect
magnesj pushed a commit that referenced this issue Dec 11, 2018
@lindkvis @magnesj
#3780 Get rid of debug output from painter
2e8c2e2 
* don't really need the boundingRect override for RiuQwtSymbol
* And it needs a painter
* Just get rid of it and pass in the painter for the labelBoundingRect
@magnesj magnesj changed the title Debug output from QPainter Remove debug output from QPainter Dec 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lindkvis lindkvis

Labels
BugInRelease Bug in an official release
Projects
None yet
Milestone
Maintenance - Patch 2018.11.1
Development

No branches or pull requests
2 participants
@lindkvis @magnesj