Minor performance improvements to MockInjector

[k163377]

SSIA

Please see the commit comments for detailed changes.

[Raibaz]

I like the usage of fold, thanks!

I don't really understand the reasoning behind moving sortCriteria to the companion object though, since it's being used only in a single place - why did you move it?

[k163377]

Because Function1[] was being instantiated on every call.
I have not checked how much this affects performance, but I believe it is an improvement.

import kotlin.reflect.KFunction

val tmp = compareBy<KFunction<Any>>(
    { -it.parameters.size },
    { fn -> fn.parameters.joinToString(",") { it.type.toString() } }
)

// decompiled
public final class TempKt {
   @NotNull
   private static final Comparator tmp;

   static {
      // `Function1[]` is generated for each call.
      tmp = ComparisonsKt.compareBy(new Function1[]{null.INSTANCE, null.INSTANCE});
   }
}

[Raibaz]

Can you please rebase on top of the master branch? I mistakenly broke the build in a previous commit :(

I fixed it now

[k163377]

Rebased

[Raibaz]

Thanks a lot for putting this together and following up!