Dmitry Khalanskiy

Results 612 comments of Dmitry Khalanskiy

A likely cause would be that you don't have the headers for curl installed on your system. For example, on Ubuntu you need `libcurl-dev`, which can be acquired with, say,...

I did encounter an assertion error when trying to query the TZDB for dates earlier than the minimum year.

Sure. With the current `master`, ```c++ current_zone()->get_info(local_seconds(seconds(-4266666666664))); ``` crashes for me with ``` src/tz.cpp:2128: date::sys_info date::time_zone::load_sys_info(std::vector::const_iterator) const: Assertion `i != transitions_.begin()' failed. ``` I encountered this on both Linux and...

Thank you! Yes, this is a bug in the test module. The fix, as is common, a single line of code.

@ephemient, not quite. `Dispatchers.Main.immediate` is a `TestDispatcher`, it just doesn't have the correct properties. Fixing #3298 would fix this particular example, but not every case. So, the issue is still...

Filed an issue to Reactor about this, as, after some research, it does look like a clear-cut case of non-compliance on their part: https://github.com/reactor/reactor-core/issues/3117 However, currently, we're also non-compliant. I...

@OlegDokuka, the interpretation that 2.7 means that `request` can not race with `request`, and `cancel` can not race with `cancel` doesn't seem plausible, because https://github.com/reactive-streams/reactive-streams-jvm#3.5 says that `cancel` must be...

> rule 3.5 is not about race conditions Could you clarify this? I'm not sure that it's not *also* about race conditions. In particular, > MUST be thread-safe where thread-safety...

I think we'll keep this open as a reminder until the spec is changed.

Good point, we should change the docs, especially given that we're dropping support for the old memory model.