Need to move anariGetObjectFeatures definition to the API.cpp implementation file about anari-sdk HOT 4 CLOSED

I guess so, for the moment they use a STB style ANARI_FEATURE_UTILITY_IMPL macro to place the defininitions. As long as only one cpp file defines that before including it, it should be fine. I guess the other option would be to just make the functions static.

I'm not sure if "hardwiring" it into the sdk is the correct path. The utility to generate this helper is intentionally written so you could create a custom one with additional (vendor or extension provided) features in it.

from anari-sdk.

It doesn't need to go in the SDK, however the current setup doesn't work when ANARI_FEATURE_UTILITY_IMPL is defined in a header file. Making them static functions may be the best solution. In the words of Jakob it just seems weird to have to define ANARI_FEATURE_UTILITY_IMPL first in order to get the list of features from a device when, presumably, all front-ends will need to do this considering the direction the spec is going.

from anari-sdk.

Workaround: I created my own GetAnariDeviceFeatures method that returns an anari::Features object that gets updated after processing a list of feature strings returned by anariGetObjectInfo.

from anari-sdk.

Closing this as the STB-style usage is the current intention and alternatives introduce link dependencies we want to avoid. Applications should only define ANARI_FEATURE_UTILITY_IMPL in a single .cpp translation unit to ensure a single definition.

from anari-sdk.