Skip to content

Rename web handles#205

Draft
madsmtm wants to merge 1 commit intomasterfrom
madsmtm/rename-web-handles
Draft

Rename web handles#205
madsmtm wants to merge 1 commit intomasterfrom
madsmtm/rename-web-handles

Conversation

@madsmtm
Copy link
Member

@madsmtm madsmtm commented Feb 28, 2026

Discussed in #185.

/// Raw display handle for the Web.
#[non_exhaustive]
#[derive(Debug, Clone, Copy, PartialEq, Eq, Hash)]
pub struct WebDisplayHandle {}
Copy link
Member Author

Choose a reason for hiding this comment

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

I just realized, an option that might make sense, even semantically, could be to include the transfer function in the display handle? WDYT?

Copy link
Member

Choose a reason for hiding this comment

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

The last update was that its almost impossible to represent a transfer function properly, regardless of where its stored.

Do you have a specific representation in mind?

Copy link
Member Author

@madsmtm madsmtm Mar 1, 2026

Choose a reason for hiding this comment

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

I was thinking that if wasm-bindgen decides that it wants multiple tables, like you write in #185 (comment), then we might be able to store that table transfer function in a WasmBindgenDisplayHandle.

Copy link
Member Author

Choose a reason for hiding this comment

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

I guess the question is also somewhat: Should we rename the WebDisplayHandle as well?

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.

2 participants