Skip to content

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

Merged
Perksey merged 2 commits into
mainfrom
patch/2.6.0-release-notes
Jul 3, 2021
Merged

Perksey merged 2 commits into
mainfrom
patch/2.6.0-release-notes

Conversation

@Perksey

@Perksey Perksey commented Jul 2, 2021

Copy link
Copy Markdown
Member

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
Copy Markdown
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
Copy Markdown
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