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.

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

Remove GL framebuffer on macOS #504

Closed
wants to merge 1 commit into from

Conversation

vespakoen
Copy link
Contributor

This is work done by @troughton (see: troughton/solvespace@42f187a).

See his comment in this issue: #354
I just wanted to turn it into a PR on this repository so more people can see it, and possibly try it out / continue working on it.

I can compile and run this, it make SolveSpace very fast again!
However, there is some flashing in the toolbar occurring now and then, and there are probably other issues that I haven't stumbled into yet.

Notice that this PR does not bring in the changes that @troughton made to src/render/gl3shader.cpp.
Those seemed to be another optimisation that I don't fully understand, and we might be able to do without.

Can someone tell me how to crash it / what needs to be done to get this working reliably?

This PR is supersedes #502

@vespakoen
Copy link
Contributor Author

From the other PR:

If you add a constraint, and double click on the text, can you edit it? Or does it disappear / become black eventually? What about scrollbars and tooltips? That's the reason it uses offscreen rendering right now.

@whitequark I don't see any of this above issues, the only thing I see that is strange is that the toolbar sometimes disappears for a frame or 2.

@whitequark
Copy link
Contributor

I don't see any of this above issues

Great! I assume they finally fixed whatever issue there was with GL layers. I'm going to recheck if this still works on Mavericks, I think.

Feel free to remove all traces of GlOffscreen, since it was only used for Gtk 2 and macOS and we have long migrated off Gtk 2.

@whitequark
Copy link
Contributor

the only thing I see that is strange is that the toolbar sometimes disappears for a frame or 2.

Is it just the toolbar? Or does selection marquee and fps counter also blink?

@whitequark

This comment has been minimized.

@whitequark
Copy link
Contributor

That was caused by a missing

self.wantsBestResolutionOpenGLSurface = YES;

@vespakoen
Copy link
Contributor Author

Cool! Shall I add it to this PR or will you do it along with the other changes you made?

@whitequark
Copy link
Contributor

I'll do it.

@vespakoen
Copy link
Contributor Author

Great, thanks a lot! it's great to have a butter smooth SolveSpace again =)

@whitequark
Copy link
Contributor

whitequark commented Nov 23, 2019

Applied as 9b61f98. Let me know if there are any issues.

@whitequark whitequark closed this Nov 23, 2019
@vespakoen
Copy link
Contributor Author

Working great on my end! Thanks a lot!

@vespakoen
Copy link
Contributor Author

vespakoen commented Nov 23, 2019

Here is a macOS build from the latest master 54015b6
I signed and notarised it using my Apple Developer account for anyone interested to check it out.

solvespace.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants