Skip to content
This repository has been archived by the owner on Jun 18, 2021. It is now read-only.

Temporarily use BackendBit::PRIMARY in examples #631

Merged
merged 2 commits into from Nov 21, 2020

Conversation

grovesNL
Copy link
Collaborator

BackendBit::all() allows GL/surfman to be used on macOS which appears to prevent rendering with Metal (e.g. hello-triangle displays an empty window).

surfman will be removed in later versions of gfx, which will allow us to switch back to BackendBit::all() in the examples.

`BackendBit::all()` allows GL/surfman to be used on macOS which appears
to prevent rendering with Metal (e.g. hello-triangle displays an empty
window).

surfman will be removed in later versions of gfx, which will allow us
to switch back to `BackendBit::all()` in the examples.
@grovesNL grovesNL requested a review from kvark November 21, 2020 03:03
@grovesNL
Copy link
Collaborator Author

cc @scoopr

Copy link

@monocodus monocodus bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an autogenerated code review.

Checker summary (by rust_clippy):
The tool has found 23 warnings, 1 errors.

The .monocodus config not found in your repo. Default config is used.
Check config documentation here

Copy link

@monocodus monocodus bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an autogenerated code review.

Checker summary (by rust_clippy):
The tool has found 23 warnings, 1 errors.

The .monocodus config not found in your repo. Default config is used.
Check config documentation here

Copy link
Member

@kvark kvark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors r+

@bors bors bot merged commit dff97b0 into gfx-rs:master Nov 21, 2020
@grovesNL grovesNL deleted the use-backend-bit-primary branch November 24, 2020 04:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants