Code Review: FunctionContext.kt

1. minor formatting issues
2. added comments
3. renamed functionByConstructor to functionsWithClosure
Code Review: JsBreak does not need to overload acceptChildren
Code Review: FunctionInlineMutator label naming changes

1. self -> this
2. use InvocationUtil.getName(call)
3. do not repeat ending `$`
4. suppress warning "MethodMayBeStatic"
Code Review: InliningContext formatting
Code Review: InlineableResult annotations
Code Review: InlinedCallExpressionTranslator

1. removed unused constructor
2. removed unused parameters
3. inlined private shouldBeInlined implementations
Code Review: moved NameCollector to util package
Code Review: FunctionInlineMutator

1. use renaming context in doReplaceReturn
2. rename doReplaceReturn -> doReplaceReturns
3. added Unit-return TODO
Code Review: Renaming.kt

1. formatting
2. removed AliasStrategy
3. use sublist instead of drop
Code Review: RenamingContext

removed unnecessary getFreshName overload
JS backend: added the ability to turn off the inline optimization.
Code Review: Collections.kt -> collectionUtils.kt
Code Review: removed unnecessary ReferenceCollector lvalue/rvalue-reference partition
Code Review: collectionUtils.kt
1. upper-cased identitySet function to be class-like
2. modified identitySet(initial) to be extension function toIdentitySet
Code Review: removed unnecessary `with`
Code Review: removed unnecessary super<> type parameter
Code Review: added assert
Code Review: renamed collectReferences function so, that it's clear that references are collected inside parameter
Code Review: use [] for Map's get/put
Code Review: replaced runtime exception with assertion