clarkmcc / cel-rust

Common Expression Language interpreter written in Rust
https://crates.io/crates/cel-interpreter
MIT License
362 stars 18 forks source link

Use Arc instead of Rc for Map type #54

Closed lucperkins closed 2 months ago

lucperkins commented 2 months ago

This change makes the Map type more suitable for multi-threaded contexts. There may, however, be reasons I'm unaware of for using an Rc for the Map type while other types use Arc. In that case, I'd be happy to put this behind a feature flag.

lucperkins commented 2 months ago

Ah, my apologies. I was looking at an out-of-date revision. Looks like this has already been done 😄