Nabil Abdel-Hafeez

Results 242 comments of Nabil Abdel-Hafeez

There is a way to workaround this ```scala .outCodec[MyData]( ContentCodec.content( HttpContentCodec( ListMap( MediaType.application.`json` -> BinaryCodecWithSchema.fromBinaryCodec( schema.codec.JsonCodec.schemaBasedBinaryCodec[MyData]( schema.codec.JsonCodec.Config(ignoreEmptyCollections = false) ) ) ) ) ) ++ StatusCodec.Ok ) ``` But it...

@countfloyd it is the most reasonable default, since it is minimal. `myList: []` is not useful, if you already know the schema. It is the most efficient way to encode.

This is a Scala 3 bug, which is fixed but not released yet. It might be back ported to 3.3.x with 3.3.7 https://github.com/scala/scala3/pull/20894?notification_referrer_id=NT_kwDOAG8jT7MxMTI5OTA1NzA3NDo3MjgzNTM1&notifications_query=repo%3Ascala%2Fscala3

Here is the [link](https://github.com/scala/scala3-lts/pull/223) to the back port PR

@andrzejressel is this issue still needed?

I am sorry, but this would require quite some internal change that has performance challenges. And it seems like a fringe use case. I don't even understand why it is...

Can you show me, how you think error handling would look like? There should not be a lot of boilerplate

Even if this works, this is not how `++` should work. Like on `Map` the right should win. If there is a bug, it is way deeper in the code

This change does not look right either. Neither the place nor the fact that we replace a while loop with allocation of an interator