https://kotlinlang.org logo
Title
a

arnaud.giuliani

02/09/2022, 3:19 PM
Checking the
includes
contribution. Seeing some perf impact, trying to see if I can optimise it a bit before releasing first 3.2.0-beta-1
m

Marcello Galhardo

02/09/2022, 3:41 PM
Hm, that is weird - we should not have any impact performance with that implementation (+ it should absolutely not affect any client that don’t use the
includes
keyword). Do you have any data to share about the impact performance? How much was it? And how many modules is your testing set? 🤔
a

arnaud.giuliani

02/09/2022, 3:42 PM
just tested it with coffeeMaker app sample in examples folder
and module loading time run from 3ms to 10ms
and nothing added in the coffeAppModule
m

Marcello Galhardo

02/09/2022, 3:46 PM
Hum. 🤔