Advertisement

Script builder addon does not detect duplicate scripts on Windows

Started by June 20, 2015 12:26 PM
6 comments, last by WitchLord 9 years, 5 months ago

The script builder addon does not detect duplicate scripts on Windows.

If the same script is added multiple times, but with differing case, it will add the same file again.

The cause is in CScriptBuilder::IncludeIfNotAlreadyIncluded, it uses a case sensitive search in the script map.

Using a case insensitive std::less replacement should suffice here.

Thanks for alerting me of this. I'll have it fixed for the next release.

AngelCode.com - game development and more - Reference DB - game developer references
AngelScript - free scripting library - BMFont - free bitmap font generator - Tower - free puzzle game

Advertisement

Just as a reminder that this should only occur on Windows, as most other operating systems are POSIX based and are case-sensitive.

Yes, I know, but thanks for the reminder anyway :)

AngelCode.com - game development and more - Reference DB - game developer references
AngelScript - free scripting library - BMFont - free bitmap font generator - Tower - free puzzle game

I've fixed this in revision 2182.

Regards,

Andreas

AngelCode.com - game development and more - Reference DB - game developer references
AngelScript - free scripting library - BMFont - free bitmap font generator - Tower - free puzzle game

scriptbuilder.cpp does not compile due to CScriptBuilder::GetSectionName using the wrong set type to declare an iterator.

Advertisement

Strange. I didn't get any compilation error for this.

I'll have it fixed. Thanks.

AngelCode.com - game development and more - Reference DB - game developer references
AngelScript - free scripting library - BMFont - free bitmap font generator - Tower - free puzzle game

Fixed in revision 2184.

AngelCode.com - game development and more - Reference DB - game developer references
AngelScript - free scripting library - BMFont - free bitmap font generator - Tower - free puzzle game

This topic is closed to new replies.

Advertisement