Skip to content

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