georust / georust/proj

`Send` and `Sync`

Open
#141 7 comments 3 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
186
Forks
63
PR merge metrics
No merged PRs in 30d

Description

What is the status about `Send` and `Sync`?
There is no explicit unimpl note, but due to the raw-pointers, it is also not auto-impled on `Proj`.
Is it safe to move the `Proj` instance between threads and have shared calls to `.project`?

If so, what about providing the impl:
```rust
unsafe impl Sync for Proj {}
unsafe impl Send for Proj {}
```

If not, what about clarifying it in the docs / README.

Background: I would like to load and parse the proj-definition once and call the projection from different places in the application.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the README and the `Proj` API described in the issue, then trace how its raw pointers are managed and whether concurrent `.project` calls are supported. Done means reaching a clear safety decision and either providing the requested `Send`/`Sync` behavior with tests or documenting the restriction.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend-api-design
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.