Skip to content

Separate the builder attributes into multiple substates#603

Merged
tomaka merged 12 commits into
masterfrom
gl-attribs
Sep 21, 2015
Merged

Separate the builder attributes into multiple substates#603
tomaka merged 12 commits into
masterfrom
gl-attribs

Conversation

@tomaka
Copy link
Copy Markdown
Contributor

@tomaka tomaka commented Sep 20, 2015

This PR removes the BuilderAttribs struct and replaces it with GlAttributes, WindowAttributes and PixelFormatRequirements.

The purpose of this change is to avoid passing windowing-related attributes to WGL/GLX/EGL, and get a better isolation.

Small breaking change: the HeadlessRendererBuilder now takes a lifetime parameter to store the context to share lists with.

cc #516
cc #517

@tomaka
Copy link
Copy Markdown
Contributor Author

tomaka commented Sep 20, 2015

Ah, I guess what I'm trying to do is not possible because of that cancer that is OS/X.

@tomaka tomaka force-pushed the gl-attribs branch 3 times, most recently from 85ac7fd to e5063c3 Compare September 21, 2015 08:25
@tomaka tomaka changed the title Extract GlAttributes from BuilderAttribs Separate the builder attributes into multiple substates Sep 21, 2015
@tomaka tomaka force-pushed the gl-attribs branch 5 times, most recently from d57a386 to 07640e7 Compare September 21, 2015 11:48
@tomaka
Copy link
Copy Markdown
Contributor Author

tomaka commented Sep 21, 2015

cc @glennw

I'm not sure if this compiles for android (the changes were made blindly). I'm going to land this anyway and the compilation will eventually be fixed in master.

tomaka added a commit that referenced this pull request Sep 21, 2015
Separate the builder attributes into multiple substates
@tomaka tomaka merged commit 6787f1d into master Sep 21, 2015
@tomaka tomaka deleted the gl-attribs branch September 21, 2015 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant