Change Defines & BaseDefines from TStringList to TLapeStringMap

nielsAD / lape

Scripting engine with Pascal-like syntax for FPC and Delphi

118 stars 28 forks source link

Change Defines & BaseDefines from TStringList to TLapeStringMap #127

Closed ollydev closed 5 years ago

ollydev commented 5 years ago

Fixes #122

nielsAD commented 5 years ago

How much of an impact does this have on script compilation times?

ollydev commented 5 years ago

Not exactly a great test but this is compiling SRL times:

defines-fix branch:

Compiled successfully in 547 ms.
Compiled successfully in 610 ms.
Compiled successfully in 609 ms.
Compiled successfully in 593 ms.
Compiled successfully in 625 ms.

master branch:

Compiled successfully in 594 ms.
Compiled successfully in 625 ms.
Compiled successfully in 625 ms.
Compiled successfully in 610 ms.
Compiled successfully in 609 ms.

Near to nothing in it?

nielsAD commented 5 years ago

Thanks!