Skip to content

Even easier Immediates#583

Open
Vipitis wants to merge 4 commits intogfx-rs:trunkfrom
Vipitis:immediates
Open

Even easier Immediates#583
Vipitis wants to merge 4 commits intogfx-rs:trunkfrom
Vipitis:immediates

Conversation

@Vipitis
Copy link
Copy Markdown

@Vipitis Vipitis commented Apr 16, 2026

Checklist

  • cargo clippy reports no issues
  • [?] cargo doc reports no issues
  • cargo deny issues have been fixed or added to deny.toml
  • Human-readable change descriptions added to CHANGELOG.md under the "Unreleased" heading.
    • If the change does not affect the user (or is a process change), preface the change with "Internal:"
    • Add credit to yourself for each change: Added new functionality. @githubname

Description

While doing the v29 update for the python wrapper. I spotted that the descriptor already has a immediateSize field in webgpu.h webgpu-native/webgpu-headers#529
Also saw the limits being simplified in #575 so this makes it even simpler, although you still need the native only feature. I updated the example and it still works, please let me know if the changes to the wgpu.h headers are fine like this.

Related Issues

@lisyarus
Copy link
Copy Markdown
Contributor

lisyarus commented Apr 16, 2026

Nice! I think the changelog diff is a bit incorrect: it marks as released in v29 everything currently marked as unreleased, which isn't the case (there were new additions after v29). Apparently the current changelog in trunk contains both pre-v29 and post-v29 changes. Maybe just add your changes to unreleased for now, and then a separate PR would sort out the changelog itself?
@cwfitzgerald

@Vipitis
Copy link
Copy Markdown
Author

Vipitis commented Apr 16, 2026

yeah, I quickly looked at the blame and all your recentt changes should probably move to the top as well. I just tried to have mine a little more correct for now. I can add a commit to move yours tomorrow, or maybe do it outside this PR?

@lisyarus
Copy link
Copy Markdown
Contributor

lisyarus commented Apr 17, 2026

I'd suggest fixing the changelog in a separate PR, just to keep the history clean?

Comment thread ffi/wgpu.h
/** Identifies @ref WGPUNativeLimits. */
WGPUSType_NativeLimits = 0x00030002,
/** Identifies @ref WGPUPipelineLayoutExtras. */
WGPUSType_PipelineLayoutExtras = 0x00030003,
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

as this enum is removed, do all the others need to move up?

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.

3 participants