linebender / linebender/piet

remove dwrote dependency

Open
#261 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement piet-direct2d
Dominant language
Rust
Stars
1.4k
Forks
98
PR merge metrics
No merged PRs in 30d

Description

in #255 we added a dependency on the dwrote crate. raph has some safety concerns there, and longer term it probably makes sense for us to implement this little bit of code ourselves; but since the windows crate ecosystem is a bit in flux right now (in particular [`com-rs`](https://github.com/microsoft/com-rs) is being iterated on heavily, and should become the standard way of implementing com interfaces) I think it makes sense to hold off on that work until things settle down a little bit.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading issue #255 to understand where the dwrote dependency was introduced. Review the current com-rs ecosystem and the safety concerns mentioned here; the work is complete only when dwrote is no longer needed and the relevant Windows text functionality still works, but this issue says to wait for the ecosystem to settle.

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
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.