adjustModuleCompleter: Avoid capturing Context

Previously we computed the scope in `findModuleBuddy` using
`this.effectiveScope`, this means that we captured `this` which has a
self-type of `Context`, replacing it by `ctx.effectiveScope`
would be wrong since we are interested in the scope at the time
`adjustModuleCompleter` was called, not the scope at the time the
completer is called. Therefore, we have to eagerly compute
`this.effectiveScope` so that we don't capture the Context but can use
the right scope in `findModuleBuddy`.

We also move `findModuleBuddy` to a companion object to avoid accidental
captures of `this`.

This capture lead to crashes in the IDE.

0 files changed, 0 insertions, 0 deletions