Skip to content

Conversation

@richerfu
Copy link

@richerfu richerfu commented Nov 21, 2025

Render successfully. :)

PureX 6.0.0.112

e1436e1c1459d7cc4f640d4a5d0b8082

Copy link
Collaborator

@waywardmonkeys waywardmonkeys left a comment

Choose a reason for hiding this comment

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

I know this is a work in progress / draft.

Overall, this makes me feel like we should clean up and improve our example code.

Is there a timeline on getting the support in winit landed? Will there be a 0.30 release with support or will it end up going into 0.31?

}

#[cfg(not(target_os = "android"))]
#[cfg(not(any(target_os = "android", target_env = "ohos")))]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this actually necessary here? IIRC, this stuff was a workaround for the multiple shaders being slow to compile ... I'd rather just not have this in here perhaps, but if we don't need to replicate this for ohos, that'd be good.

Copy link
Author

Choose a reason for hiding this comment

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

I'm not sure it. But i think it's not bad to keep same with android now. :) May we can remove it later.

@richerfu
Copy link
Author

richerfu commented Nov 26, 2025

I know this is a work in progress / draft.

Overall, this makes me feel like we should clean up and improve our example code.

Is there a timeline on getting the support in winit landed? Will there be a 0.30 release with support or will it end up going into 0.31?

I hope 0.31 can support ohos. I'm trying to do it. :)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants