BaseMax / BaseMax/MiniArchiveCompressor

A few small tips

Open
#7 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
QML
Stars
11
Forks
2
PR merge metrics
No merged PRs in 30d

Description

Id doesn't match with function in estimateTime and estimateSize.
Text is better write with qstr for use multi language and other uses.
Use relative addressing instead of absolute addressing.

@BaseMax

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate the estimateTime and estimateSize functions, then trace the text and addressing code mentioned in the issue. Confirm the identifier matches its function, use qstr where multilingual text is involved, and replace absolute addressing with relative addressing where applicable.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
desktop
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
32/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.