[Suggestion] String implicit cast warning

[Mikk155]

i was a bit of annoyed by the warning:

WARNING: Angelscript: Float value truncated in implicit conversion to integer

But then i stoped to think out. the compiler doesn't warn about string conversion.

I have seen horrible codes which are endless else-if conditions creating countless of strings from implicit conversion for Sven Co-op's string_t.

I believe the same float/int warning applied here will be of help to tell new scripters to instantiate a string in the scope instead of doing that string_t (Or any other implicit casting) multiple times.

[codecat]

I think this is an implementation detail. In your case, Sven Coop could implement this themselves either by simply not supporting a string constructor with parameters of other types, or explicitly warning when they are being used.

That said, it could be nice to have the ability to tell the script engine about specific flags that the compiler will warn about when used. I could see this being useful for deprecated APIs, for example. Not sure what the API for that would look like though.

[anjo76]

I don't know how SvenCoop is using AngelScript so I cannot comment on their implementation.

But a way to disable compilers warnings in AngelScripts is certainly doable. I have this on the to-do list already here https://docs.google.com/document/d/1st0NAN_zQkiigxWCTEbLvpwNyYE1Q91LeGchFq8mWAw/edit?tab=t.28y4c7vy6faf#heading=h.ghy5yxqofgqm

Adding custom warnings is a bit more complicated though, as it involves defining the hooks in the compiler where a custom callback can be called to generate the warnings.