Wrong parameter type in index.d.ts

[maierfelix]

Wrong parameter type in index.d.ts

[maierfelix]

nvk's documentation is generated based on the original documentation and not finished yet - the documentation incorrectly states here that VkBool32 has to be used, but VkInout is the correct type.

Edit: Having a quick look at the type definition file, it seems that VkInout should also allow boolean values.

[norman784]

But if you use VkInout it doesn't work, if I use this code

let surface_support = { $: 0 }
vkGetPhysicalDeviceSurfaceSupportKHR(this.physical_device, 0, this.surface, surface_support)

I get

vkGetPhysicalDeviceSurfaceSupportKHR(this.physical_device, 0, this.surface, surface_support)                                                              
    ^                                                                          
TypeError: Expected 'Boolean' for Object property '$' 4 'pSupported' in 'vkGetPhysicalDeviceSurfaceSupportKHR' 

And if use this

let surface_support = { $: false }
vkGetPhysicalDeviceSurfaceSupportKHR(this.physical_device, 0, this.surface, surface_support)

I get

Argument of type '{ $: boolean; }' is not assignable to parameter of type 'VkInout'.
  Types of property '$' are incompatible.
    Type 'boolean' is not assignable to type 'number'. 
vkGetPhysicalDeviceSurfaceSupportKHR(this.physical_device, 0, this.surface, surface_support)  
                                                                                                                           ~~~~~~~~~~~~~~~  

I'm not sure another way to solve this issue rather than change in the generated files the VkInout for VkBool32

[maierfelix]

Hey my comment edit above was a reminder for myself, not an actual answer :) - this is a bug in nvk

I'm busy this week, but if you can't wait for a fix, then (depending on your OS) you can try editing the two top lines of this file, by changing them from this:

declare var VkInout: { $: number; }
export interface VkInout { $: number; }

to this:

declare var VkInout: { $: number | boolean; }
export interface VkInout { $: number | boolean; }

[norman784]

Cool, thanks!

[maierfelix]

This should be fixed in v0.6.3

[norman784]

I don't know if is related, but the v0.6.3 points to the generated files v1.1.108 instead of the bundled one v1.1.114.

[maierfelix]

@norman784 sorry for that, is v0.6.4 working for you now?

[norman784]

Thanks, now it works!