`layerName` isn't optional

FacticiusVir / SharpVk

C# Bindings for the Vulkan API & SPIR-V

MIT License

147 stars 18 forks source link

`layerName` isn't optional #51

Closed Y-Less closed 5 years ago

Y-Less commented 5 years ago

Instance.EnumerateExtensionProperties and PhysicalDevice.EnumerateDeviceExtensionProperties take an optional pLayerName, but there's no default null on the parameter:

        public unsafe SharpVk.ExtensionProperties[] EnumerateDeviceExtensionProperties(string layerName)

FacticiusVir commented 5 years ago

In the .partial files that's a manual change I can make directly; I know there's also a few cases in the generator that don't detect optional inputs so I'll check this for the generated code.