mirrors/bgfx
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix random, erroneous D binding type generation; add missing IDL defaults (#3210)

Browse Source 
* Reformatted comments; fixed a couple of oversights

* D bindings: deterministic sub-struct order

* Added missing default to IDL

* Fixed sub-struct linkage; regenerate D binds

* Culled D bindings for header-only C++ functions

* Added missing default to bgfx.idl

* cppinline now supported by all auto-gen bindings

The pattern "func.cppinline and not func.conly" is to make sure that C bindings for `bgfx_vertex_layout_has` are still generated.

* Fix mangling issue; use updated BindBC-Common API

* Add missing default to setTransform in IDL

* Fix erroneous generation of `uc_int64`

Non-deterministic ordering of hash-maps were the culprit all along!

* Add missing default to overrideInternal IDL & re-generate
This commit is contained in:
IchorDev 2023-12-03 01:19:45 +07:00 committed by GitHub
parent 7af65cbbfc
commit ae4b0cd5a9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 12 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
bindings/d/package.d
View File

@ -3078,7 +3078,7 @@ mixin(joinFnBinds((){
- `BGFX_SAMPLER_[MIN/MAG/MIP]_[POINT/ANISOTROPIC]` - Point or anisotropic
sampling.
*/
{q{size_t}, q{overrideInternal}, q{TextureHandle handle, ushort width, ushort height, ubyte numMIPs, bgfx.fakeenum.TextureFormat.Enum format, c_uint64 flags}, ext: `C++, "bgfx"`},
{q{size_t}, q{overrideInternal}, q{TextureHandle handle, ushort width, ushort height, ubyte numMIPs, bgfx.fakeenum.TextureFormat.Enum format, c_uint64 flags=Texture.none | Sampler.none}, ext: `C++, "bgfx"`},
/**
* Sets a debug marker. This allows you to group graphics calls together for easy browsing in
@ -3160,7 +3160,7 @@ mixin(joinFnBinds((){
mtx = Pointer to first matrix in array.
num = Number of matrices in array.
*/
{q{uint}, q{setTransform}, q{const(void)* mtx, ushort num}, ext: `C++, "bgfx"`},
{q{uint}, q{setTransform}, q{const(void)* mtx, ushort num=1}, ext: `C++, "bgfx"`},
/**
* Set model matrix from matrix cache for draw primitive.

2
scripts/bgfx.idl
View File

@ -2715,6 +2715,7 @@ func.overrideInternal { cname = "override_internal_texture" }
--- mode.
--- - `BGFX_SAMPLER_[MIN/MAG/MIP]_[POINT/ANISOTROPIC]` - Point or anisotropic
--- sampling.
{ default = "BGFX_TEXTURE_NONE | BGFX_SAMPLER_NONE" }
-- Legacy API:
@ -2795,6 +2796,7 @@ func.setTransform
--- to be used for other draw primitive call.
.mtx "const void*" --- Pointer to first matrix in array.
.num "uint16_t" --- Number of matrices in array.
{ default = 1 }
--- Set model matrix from matrix cache for draw primitive.
func.setTransform { cname = "set_transform_cached" }

16
scripts/bindings-d.lua
View File

@ -228,20 +228,20 @@ local function convArray(array)
end
local typeSubs = {
uint32_t = "uint", int32_t = "int",
uint16_t = "ushort", int16_t = "short",
uint64_t = "c_uint64", int64_t = "c_int64",
uint8_t = "ubyte", int8_t = "byte",
uintptr_t = "size_t"
{"uint32_t", "uint"}, {"int32_t", "int"},
{"uint16_t", "ushort"}, {"int16_t", "short"},
{"uint64_t", "c_uint64"}, {"int64_t", "c_int64"},
{"uint8_t", "ubyte"}, {"int8_t", "byte"},
{"uintptr_t", "size_t"}
}
local function convSomeType(arg, isFnArg)
local type = arg.fulltype
if type == "bx::AllocatorI*" or type == "CallbackI*" then
type = "void*"
else
for from, to in pairs(typeSubs) do
if type:find(from) then
type = type:gsub(from, to)
for _, item in ipairs(typeSubs) do
if type:find(item[1]) then
type = type:gsub(item[1], item[2])
break
end
end