spelling mistake: GetGategory should be GetCategory

[GrahamAsher]

This affects teascript::runtime_error::GetGategory, which should be renamed GetCategory.

[Florian-Thake]

Thank you very much @GrahamAsher for reporting this issue!

This is a stupid, ugly typo, which slipped through. Sorry for that, it must be fixed for sure.

The problem is, this is not only a cosmetic issue. The function is part of the public API and on top of that, it is also a virtual function.

I need to think about whether a deprecated forwarding function or a temporary opt-in macro for backward compatibility makes sense here, as I did before with other breaking changes.
Also, the fix will then come with an entry in Deprecation_and_Breaking_Changes.txt and in changelog.txt.

I will update this issue when the fix is there. Thanks again for taking the time to report it.

[GrahamAsher]

You're welcome! But I don't think this breaking change is very important in terms of user inconvenience.

There's a high probability that no-one yet uses the function, or at least that no-one has yet derived any classes from it, because to a native English speaker the typo sort of 'jumps out' and it would have been reported to you before. But in any case it's trivial for users to recompile. They won't even have to recompile if they have incorporated the TeaScript headers into their projects, as I have, until they decide to upgrade. And then a simple find-and-replace fixes everything.

I have now incorporated TeaScript into the development branch of CartoType as its official scripting language and I'm in the middle of testing it and the way it connects to the rest of CartoType. I commend your decision to make it a header-only library, which makes everything much easier.

[Florian-Thake]

Yes, maybe the likelihood is small that it will hit somebody, but it is not zero.

I will keep my promise from the README:

[...] usage of the high-level C++ API only. This API will stay backward compatible or a soft migration will be provided – except if major issues are detected or at very rare circumstances. The high-level API consists of the classes teascript::Engine / teascript::EngineBase, all public getters in teascript::ValueObject as well as everything in Exception.hpp / SourceLocation.hpp and version.h (except if otherwise noted).

I will credit you in changelog.txt as the reporter arriving together with the fix commit.

I don't know about CartoType yet. If you like you can provide a link and/or let me know what are the requirements and use cases for a script language from the perspective of CartoType. I am pleased that you decided to incorporate it. Let me know how it is going if you like.