Engine devs agreed that there aren't 'unsafe' changes and if there are than they are engine bugs. So I would like to remove that function in the next Spring RC. Opinion by modders welcome, if this is too short dated and you already fixed your mods with current git.
Engine devs agreed that there aren't 'unsafe' changes and if there are than they are engine bugs. So I would like to remove that function in the next Spring RC. Opinion by modders welcome, if this is too short dated and you already fixed your mods with current git.
This is not clear, do you mean that AllowUnsafeChanges("USE AT YOUR OWN PERIL") will always be on or do you mean that you want to remove the ability to give orders outside of gameframe and so on?
since this is a quite important change, mod-breaking even, may I suggest creating another sticky in the mod section like for the other compatibility-breaking changes?
also, given the short notice ( other changes had 6 months+ warning, and were delayed by 1 major release too ) I think it would be better than instead of removing the function completely, to add an empty stub in the engine that prints a deprecation warning, and remove the stub in the next major release.
the 2 proposals are distinct, while you can bring the "break all at once" argument for the second point, there's no reason not to advertise the change properly like the others were
also, given the short notice ( other changes had 6 months+ warning, and were delayed by 1 major release too ) I think it would be better than instead of removing the function completely, to add an empty stub in the engine that prints a deprecation warning, and remove the stub in the next major release.
I agree with that, deprecation warnings given out by the engine are probably the best way to communicate with the developer. No need to break things more than necessary.
Joined: 22 Feb 2006, 01:02 Location: cheap kitchen
moving to archive. I think the place where most people got in contact with AllowUnsafeChanges was the old game_spawn.lua but now it has been updated since a while: https://github.com/spring/spring/blob/m ... _spawn.lua
Users browsing this forum: No registered users and 4 guests
You cannot post new topics in this forum You cannot reply to topics in this forum You cannot edit your posts in this forum You cannot delete your posts in this forum You cannot post attachments in this forum