Skip to content

Commit d77db4b

Browse files
committed
Take the release section out of the TODO
Both entries under it are done: corecmd is ported and validated by qmsetup's own CI, which pins this HEAD as its submodule, and v1.1.0.0 is tagged with a changelog beside it. processMemoryUsage() goes on the list of gaps in their place. The coverage sweep found it has never had a caller and nothing can give it one, being static and in no header, so it is a decision rather than a test that is missing.
1 parent eb9aaaf commit d77db4b

1 file changed

Lines changed: 2 additions & 6 deletions

File tree

docs/TODO.md

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,12 @@
11
# Status
22

3-
Usable, and used. The version says 1.1, and what that promises is written down: the headers stay source compatible across a major version, and the soname carries the minor as well, so a minor bump is free to change what the binary exports.
4-
5-
## Before a release
6-
7-
- Port qmsetup's `corecmd` onto `cli`. It is the one consumer this was written for, and until it builds against it the API has been validated by nothing but its own tests.
8-
- Nothing has ever been tagged, and there is no changelog. Both wait on a stretch with no breaking change in it, which the reader API of `ParseResult` and the signature of `VersionNumber::fromString` have not had yet.
3+
Released as v1.1.0.0, and used by qmsetup's `qmcorecmd`. What the version promises is written down: the headers stay source compatible across a major version, and the soname carries the minor as well, so a minor bump is free to change what the binary exports.
94

105
## Known gaps
116

127
- The registry code assumes a little-endian host
138
- `support/commandline.h` is over a thousand lines of inline code, paid for by every translation unit that includes it
9+
- `processMemoryUsage()` in `src/system.cpp` has never had a caller. It is `[[maybe_unused]] static`, is declared in no header, and carries `<Psapi.h>` and `<mach/mach.h>` in with it. Either delete it or promote it to `system::` with a comment and a case, since nothing can cover it as it stands.
1410

1511
## Wanted
1612

0 commit comments

Comments
 (0)