novelrt / NovelRT

A cross-platform 2D game engine accompanied by a strong toolset for visual novels.
MIT License
183 stars 43 forks source link

Add minimal support for `ResourceManagement` namespace in the C API. #518

Closed RubyNova closed 1 year ago

RubyNova commented 1 year ago

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) This PR introduces a skeleton-level C API support for the ResourceManagement namespace.

It also fixes a typo in the internal error message functions and a deprecated header.

Is there an open issue that this resolves? If so, please link it here. No - it is intended to aid in ublocking #499.

What is the current behavior? (You can also link to an open issue here) There is no C API for this.

What is the new behavior (if this is a feature change)? There is now a minimal C API that should, in theory, work. Might require some of @jeuxjeux20 's PRs to be fully functional. Again this is only skeletal.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?) No.

Other information: There are missing tests due to the fact that this is wrapping a plugin API. This means that until the C API actually has a way to obtain a plugin, we can't write the relevant tests for some things. I've added tests where I can.