[PROPOSAL] Test every command-line option, including what the C code receives
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 42/100
Direzione di ricerca
Start by reading the existing argument tests from #2041 and the Rust parser in args.rs; trace how ccxr_parse_parameters passes settings into the C ccx_options structure. The proposal is split into small PRs: add a helper for checking C values, detect options without tests, then cover untested options and legacy spellings. Done means these paths and options have regression tests; coordinate with the author, who says they plan to work on it.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The Sample Platform NG page says: "We are not testing every possible argument... We broke some things on the rust migration that apparently flew under the radar."
Since the Rust argument parser replaced the C one, a number of options have broken without any test noticing:
- old spellings rejected (#2367), negative
--delay(#2370),ccextractor.cnfignored (#2369) --sentencecap/--kfmissing most words (#2397)- bad time values crashing (#2374, #2386),
--sendto/--buffersize(#2393) - still on master:
--defaultcolor,-smi,--streamtype 0x24, combined debug flags (fixes ready)
Why the current tests (#2041) didn't catch these:
- About 38 of 171 options have no test at all, e.g.
--defaultcolor,--smi,--stream,--stdin,--udp,--tcp. - No test checks what the C code receives. The tests check the Rust
Optionsafter parsing, but bugs like--defaultcolor(wrong C type) and--sentencecap(list not sorted for C'sbsearch) happen when the settings are handed to C. - Most tests skip the real entry path (
ccxr_parse_parameters, including the legacy-spelling conversion), which is how--delay -500went unnoticed.
I'd like to add, in small PRs:
- A test helper that runs options through the real path (
ccxr_parse_parameters→ Cccx_options) and checks the C values. - A test that fails when an option in
args.rshas no test, so new options can't be added untested. - Tests for the untested options, plus the old v0.94 spellings.
Does this direction sound good? i will work on this
- Lingua principale
- C
- Stelle
- 908
- Fork
- 595
- Merge medio
- 3g 23h
- PR unite (30g)
- 32
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di CCExtractor/ccextractor
-
[BUG] Filename sequence expansion (`+`) panics on numeric values exceeding `i32::MAX`Forse già presa @muntasir-396 l’ha presa 4 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
CCExtractor/ccextractor#2402 ·
I maintainer di solito rispondono entro 1 giorno
-
CCExtractor: stack out-of-bounds write and quadratic out-of-bounds read in the DVB subtitle descriptor parser (parse_dvb_description)Forse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
CCExtractor/ccextractor#2417 ·
I maintainer di solito rispondono entro 1 giorno
-
[BUG] --out=report: heap-use-after-free in process608(), and a crash with more than one input fileForse già presa @Deepak-negi11 l’ha presa 2 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
CCExtractor/ccextractor#2412 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
CCExtractor/ccextractor#2398 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 62/100
CCExtractor/ccextractor#2379 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di CCExtractor/ccextractor
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
polhenarejos/pico-fido#302 ·
-
开发板墙缺少 ESP32_S31_MOSAICOAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
espressif/esp-vision#16 · 1 commento ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
libsdl-org/SDL#16577 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
I maintainer di solito rispondono entro 1 giorno
-
halt_error adds a trailing newline for non-string input, contrary to the manualForse già presa @Huang-404-Q l’ha presa oggi. Aperta
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 2 giorni