You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
OFX::Host::ImageEffect::Base::supportsTiles returns an incorrect value when kOfxImageEffectPropSupportsTiles is set on the plugin rather than the instance
#177
Plugins can declare support (or not) for tiles by setting kOfxImageEffectPropSupportsTiles on either the plugin descriptor or the plugin instance (per the docs).
If a plugin explicitly sets the property to false on the plugin descriptor (in kOfxActionDescribe ) but sets no value on the instance (in kOfxImageEffectActionDescribeInContext) then supportsTiles returns true even though the plugin has declared that it does not support tiling.
Expected behaviour
If a plugin sets kOfxImageEffectPropSupportsTiles to false on the plugin descriptor, then OFX::Host::ImageEffect::Base::supportsTiles should return false.
Actual behaviour
If a plugin sets kOfxImageEffectPropSupportsTiles to false on the plugin descriptor, then OFX::Host::ImageEffect::Base::supportsTiles returns true.
Notes
I believe this is because a default is declared in the instance property set, and this default overrides the value that would otherwise be inherited from the plugin descriptor. I think it would be more correct for kOfxImageEffectPropSupportsTiles to be treated the same way as kOfxImageEffectInstancePropSequentialRender, and set the initial value to the one from the plugin descriptor.
The text was updated successfully, but these errors were encountered:
Problem
Plugins can declare support (or not) for tiles by setting
kOfxImageEffectPropSupportsTiles
on either the plugin descriptor or the plugin instance (per the docs).If a plugin explicitly sets the property to
false
on the plugin descriptor (inkOfxActionDescribe
) but sets no value on the instance (inkOfxImageEffectActionDescribeInContext
) thensupportsTiles
returnstrue
even though the plugin has declared that it does not support tiling.Expected behaviour
If a plugin sets
kOfxImageEffectPropSupportsTiles
tofalse
on the plugin descriptor, thenOFX::Host::ImageEffect::Base::supportsTiles
should returnfalse
.Actual behaviour
If a plugin sets
kOfxImageEffectPropSupportsTiles
tofalse
on the plugin descriptor, thenOFX::Host::ImageEffect::Base::supportsTiles
returnstrue
.Notes
I believe this is because a default is declared in the instance property set, and this default overrides the value that would otherwise be inherited from the plugin descriptor. I think it would be more correct for
kOfxImageEffectPropSupportsTiles
to be treated the same way askOfxImageEffectInstancePropSequentialRender
, and set the initial value to the one from the plugin descriptor.The text was updated successfully, but these errors were encountered: