drawFittedText has gone for a walk?
-
It was there last week and now it's gone poof!
@Christoph-Hart Was this intentional? And if not do we blame the AI? And if we blame the AI has it caused any other issues?
-
@David-Healey Oof! It got swallowed up in this commit
Shouldn't we always blame humans? Because everything should be looked over before committing.
-
haha actually that was one of the few lines of code that I wrote myself - I was testing out some new logic that catches deprecations but somehow I deleted that line.
Funnily the documentation pipeline that I'm currently running over the entire codebase to recreate the API docs flagged this as an issue:
Missing registration: drawFittedText has a wrapper (API_VOID_METHOD_WRAPPER_5) and full implementation but does NOT appear in the constructor's ADD_API_METHOD calls. It is present in the base JSON, meaning the Doxygen-based generator picks it up from the header declaration. Whether it is actually accessible at runtime is uncertain -- it may be silently unavailable if the ADD_API_METHOD is required for the scripting engine to find the method.
But it then got lost in the noise - I have a system that populates a list of bugs / small issues that the doc generator discovers while chewing through the code, but here it didn't appear too:
https://github.com/christoph-hart/hise_api_generator/blob/main/enrichment/issues.md