Implement the ImagePixelGetter trait

Open
#35 4 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
35/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Stale
Tech stack
rust
Domain
embedded-iot

Research direction

Review PR #34 and the existing pixel getter methods on Bmp and RawBmp. Check the embedded-graphics version that introduces ImagePixelGetter, then verify the trait implementation and confirm that the existing pixel access works through it for both types.

Written by the indexing model from the issue text.

Description

PR #34 did add a pixel getter method to Bmp and RawBmp, but didn't yet implement the e-g ImagePixelGetter trait. The Bmp::pixel method should be moved to a ImagePixelGetter trait impl when a new e-g version with this trait is released.

Dominant language
Rust
Stars
34
Forks
13
PR merge metrics
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

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.

More from embedded-graphics/tinybmp

All issues in embedded-graphics/tinybmp

Similar issues

More Rust issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.