Making Logger unwind safe

[jsgf]

Right now trying to use a Logger within a std::panic::catch_unwind block fails to compile with

the trait `std::panic::RefUnwindSafe` is not implemented for `slog::ser::SyncMultiSerialize + 'static`

I can use AssertUnwindSafe, but I'm not sure if that's actually true.

Is there currently a way to use slog safely in this case, or if not, could there be?

[dpc]

I need to do some homework on uniwind safety. IIRC, it's about adding a marker trait here and there if types are indeed unwind safe. I think all serializable key&values that slog supports should be.

Thank you for reporting it and please remind me in case I don't get back to you until the end of the weekend.

[dpc]

So they way I understand it, I'll have to add UnwindSafe bound on some of the traits that users can implement for key-value pairs.

I'm in the middle of some other stuff, but let's keep it open.

[dpc]

This will be fixed in slog v2 (actually next alpha release of it). Please test at first opporunity (when all drains you need will be ported to slog v2).

[dpc]

slog v2.0.0-alpha2 has been released. There are some compiler regressions (see #115) and it's generally PITA because all trait objects now require a UnwindSafe qualifier which sometimes needs to be done through a separate trait, but oh well...

Closing. Feel free to reopen in case of troubles.