georust / georust/earcut

Consider removing `unsafe`

Open
#35 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
58
Forks
5
PR merge metrics
No merged PRs in 30d

Description

There are a few bits of code that use `unsafe` keyword. I wonder if we can rework it somehow to preserve performance but be fully "safe rust"? In the mean time, we should at least document why we think that code is safe.

Contributor guide

No contributing guide indexed for this repository

Research direction

Search the repository for each use of the Rust unsafe keyword and inspect the surrounding geometry and triangulation code. Determine whether each use can be replaced without losing performance; otherwise document why it is safe. Done means the unsafe sections are removed or their safety rationale is recorded.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
computer-graphics
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.