Spring RTS Engine

in "rts\Sim\Units\Scripts\LuaScriptNames.cpp"
scriptNames[LUAFN_SetSFXOccupy] = "setSFXoccupy";

needs to be:
scriptNames[LUAFN_SetSFXOccupy] = "SetSFXOccupy";

Needs correction for consistency.

I want to order 3 kitchens?


can I has delivery soon?

Was noticed before:
http://springrts.com/phpbb/viewtopic.ph ... py#p417643
when discussed in chat iirc the result was "oh no all the scripts needs updating!" and that the real problem is the silly name and behaviour of that function.

If the name gets changed and script need to be adjusted anyway, then please not just for making small s into big S. Instead sensible name like TerrainTypeChange() or something.

Well not just the S needs correction.

Still whatever, make it a new name, just make it a better, consistent name. COB/BOS needn't change but IMO internally and Lua needs change.

Don't rename functiuons just for the sake if renaming, it breaks backward and forward compatibility and besides, the function works just as well even if it has a silly name or not. It's like wondering how that hamburger looks like when you have eaten it. It's better not to.

how many lua unit scripts would this break?

Maybe a few.

I was very clear that they should update the Lua only.

I stand corrected

So the engine developers never responded to this.

knorke wrote:Was noticed before:
http://springrts.com/phpbb/viewtopic.ph ... py#p417643
when discussed in chat iirc the result was "oh no all the scripts needs updating!" and that the real problem is the silly name and behaviour of that function.

that