Change internal representation of commands into an `enum`
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 55/100
Research direction
Start by locating the existing Command struct and all code that reads its code field. Update command representation and its users to match the proposed Command enum and CommandDetails structure. Done means commands are represented by enum variants with their associated details, without storing the command character in Command.
Written by the indexing model from the issue text.
Description
Rather than storing inside Command the command's character as code, represent each command as an enum.
So the existing:
pub struct Command {
pub code: char, // Command code
pub addr1: Option<Address>, // Start address
will become
// Sed commands derived from single-character command codes
pub enum Command {
Append, // a
Branch, // b
Change, // c
// …
}
pub struct CommandDetails {
pub command: Command, // Command to perform
pub addr1: Option<Address>, // Start address
- Dominant language
- Rust
- Stars
- 110
- Forks
- 36
- Avg merge
- 1d 23h
- Merged PRs (30d)
- 30
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from uutils/sed
-
Difficulty 3/5 1-2 days Newbie friendliness 55/100
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 48/100
Maintainers usually reply within 1 day
-
l command: the wrap width comes from the terminal size, and -l is ignoredPossibly taken @SichenLiang claimed this 43 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 35/100
Maintainers usually reply within 1 day
-
Back-references are rejected in byte (C locale) modePossibly taken @Omkar-Wagholikar claimed this 7 days ago. Opengood first issue
Difficulty 4/5 3-5 days Newbie friendliness 65/100
Maintainers usually reply within 1 day
-
s/// replacement: implement the \U \L \u \l \E case-conversion escapesPossibly taken @dedsec-terminal claimed this 35 days ago. Opengood first issue
Difficulty 3/5 1-2 days Newbie friendliness 68/100
Maintainers usually reply within 1 day
Similar issues
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
metasequoiaime/msime#9113 ·
Maintainers usually reply within 1 day
-
area:desktop priority:p2 type:chore
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
Maintainers usually reply within 1 day
-
area: supply-chain
Difficulty 1/5 Under an hour Newbie friendliness 73/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 67/100
-
Refresh stale docsOpen
Difficulty 1/5 1-3 hours Newbie friendliness 67/100
OpenCarControls/firmware#35 ·