Move a local function outside of the containing inline function

[lunakoly]

Local functions inside inlines were never expected to work and were allowed in K2 by accident. Until they are explicitly allowed, they should be avoided.

^KT-77986

[dkhalanskyjb]

The change makes sense. kotlin-community/k2/dev doesn't need an approval from a maintainer, though. Instead, the release team is responsible for that branch.

[qwwdfsad]

Aren't the changes supposed to go into develop and then merged into kotlin-community/k2/dev to avoid branches divergence?

[dkhalanskyjb]

(NB: kotlinx-datetime only has master) We can do it that way, yes, but it's also okay to put changes directly into k2/dev, and then, when a library upgrades to the new Kotlin that prohibits some behavior, we can introduce the change as part of the upgrade. I don't have a preference here, both are fine by me.

[FenstonSingel]

Instead, the release team is responsible for that branch.

Aren't the changes supposed to go into develop [first]?

FTR JiC: kotlinx-datetime is, in fact, the only kotlinx project that is not being maintained as a KUP by either the Release team or the Libraries team; that honor is mine. Unfortunately, it means two things: 1) I'm not extensively privy to the Release team's processes and commitments regarding kotlinx-as-KUPs (nor do I follow them to a tee as a result of that); 2) my current KF/KLE QA workload has pushed most of my KUP maintenance tasks to the backburner either way.