Nikita Sokolov

Results 44 comments of Nikita Sokolov

I don't think I remember the whole context, but the problem was that in case of `MediaType.MULTIPART_FORM_DATA_TYPE` the message body was also a stream and was being consumed twice. When...

What about using `CoroutineStart.UNDISPATCHED` inside `runBlocking` if there is no `CoroutineDispatcher` specified, so the blocks without suspensions would be done before giving chances to external tasks?

Thanks, though looking at this cop now I think of me being too petty about this code style. :)

I guess the real package name gets prefixed by `ammonite`: https://github.com/com-lihaoyi/Ammonite/blob/main/amm/compiler/src/main/scala/ammonite/compiler/DefaultCodeWrapper.scala#L20 https://github.com/com-lihaoyi/Ammonite/blob/main/amm/util/src/main/scala/ammonite/util/Util.scala#L100 https://github.com/com-lihaoyi/Ammonite/blob/89836cd882860fde0b2c6c62a562645e45f9d174/amm/interp/src/main/scala/ammonite/interp/Interpreter.scala#L236 It would be nice to make it customizable or even disable it.

I would like to mention the problems the existence of forget and ManualDrop causes in parallelism: by allowing freeing memory without calling destructors, it becomes impossible to notify the other...

Yeah, I have already found the weird examples, where though the memory is not being reclaimed without the destructors being run, but the references stored in it are becoming unborrowed.

Doubling that, the docs link for populator is empty - http://trailblazer.to/gems/representable/3.0/populator.html

I have come up with a migration from `parse_strategy` to `populator` lambda, the named method signature should look similar: ``` ruby collection :cast_members, parse_strategy: lambda do |fragment, _, options| represented...

It made me think for a while, but it is a nice one! Maybe it could have more proof in it to ease understanding.