RenderPipelineShaders icon indicating copy to clipboard operation
RenderPipelineShaders copied to clipboard

Fix Builder test update info

Open Mcgode opened this issue 1 year ago • 1 comments

In test_builder, we use the following structure to pass graph info:

struct PrivateUpdateInfo
{
    uint32_t width;
    uint32_t height;
    RpsBool  bUseOffscreenRT;
    RpsBool  bUseMSAA;
};

However we were using rpsTypeInfoInitFromType(void*) to pass param type information to the graph. This resulted in the graph param size being set to 8 bytes (the size of a pointer on an x64 machine), while the actual data in 16 bytes long. This meant that the struct data was only partially copied, and that the values of booleans fetched in the builder callback were undefined.

This PR fixes the issue by passing the correct type in the param information field.

Mcgode avatar May 09 '23 06:05 Mcgode