8000 Update release notes & bindings for Silk.NET 2.6.0 by Perksey · Pull Request #535 · dotnet/Silk.NET · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update release notes & bindings for Silk.NET 2.6.0 #535

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 3, 2021

Conversation

Perksey
Copy link
Member
@Perksey Perksey commented Jul 2, 2021

No description provided.

@Perksey Perksey added this to the 2.X milestone Jul 2, 2021
@Perksey Perksey requested a review from HurricanKai July 2, 2021 16:07
@Perksey Perksey enabled auto-merge (squash) July 2, 2021 16:08
public unsafe partial int GetSupportedGltextureFormats([Flow(FlowDirection.In)] nint context, [Flow(FlowDirection.In)] INTEL flags, [Flow(FlowDirection.In)] uint image_type, [Flow(FlowDirection.In)] uint num_entries, [Flow(FlowDirection.Out)] out uint gl_formats, [Flow(FlowDirection.Out)] uint* num_texture_formats);

[NativeApi(EntryPoint = "clGetSupportedGLTextureFormatsINTEL")]
public partial int GetSupportedGltextureFormats([Flow(FlowDirection.In)] nint context, [Flow(FlowDirection.In)] INTEL flags, [Flow(FlowDirection.In)] uint image_type, [Flow(FlowDirection.In)] uint num_entries, [Flow(FlowDirection.Out)] out uint gl_formats, [Flow(FlowDirection.Out)] out uint num_texture_formats);
Copy link
Member

Choose a reason for hiding this comment

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

gl_formats -> glFormats? Is this a new issue or have I just never noticed (changing parameter names is a breaking change)

Copy link
Member Author

Choose a reason for hiding this comment

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

This is exactly how it's defined upstream. We don't do any name modifications to parameter names.

@Perksey Perksey merged commit 1409a8e into main Jul 3, 2021
@Perksey Perksey deleted the patch/2.6.0-release-notes branch July 3, 2021 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0