Skip to content
This repository has been archived by the owner on Aug 3, 2024. It is now read-only.

Destroy on Cursor, Seat, and output::Layout should return a result #269

Open
Timidger opened this issue Feb 16, 2019 · 0 comments
Open

Destroy on Cursor, Seat, and output::Layout should return a result #269

Timidger opened this issue Feb 16, 2019 · 0 comments
Labels
Easy Ergonomics

Comments

@Timidger
Copy link
Member

Currently they don't return anything and just try to upgrade the handle and then drop it without mem::forgeting it. This is sound, but it doesn't do what it seems like: if it's currently borrowed, it will not drop it. We should indicate this by returning a Result.
@Timidger Timidger changed the title Destroy on Cursor and output::Layout should return a result Destroy on Cursor, Seat, and output::Layout should return a result Feb 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Easy Ergonomics
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Timidger