C90 Compliance
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia con sds.h e sds.c, quindi riproduci le diagnosi C90 riportate usando la modalità C90 o pedantic del compilatore. Verifica se la libreria può essere compilata senza i problemi relativi a flexible array member, inline e long long mostrati nel report; il lavoro è completato quando i sorgenti sds vengono compilati per la destinazione C89/C90 richiesta.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
I'm trying to get another library (https://github.com/redis/hiredis) to compile with an app that is shooting for C89/C90 compliance. I'm running into problems. It uses sds.
The errors in sds that are preventing compilation are things like the following:
sds.h:48:10: warning: ISO C90 does not support flexible array members [-Wpedantic]
char buf[];
sds.h:86:8: error: unknown type name ‘inline’
static inline size_t sdslen(const sds s) {
sds.c:640:52: warning: ISO C90 does not support ‘long long’ [-Wlong-long]
Is there any chance that sds (being a low level library) can be changed to become C90 compliant?
Related Issue:
https://github.com/redis/hiredis/issues/494
- Lingua principale
- C
- Stelle
- 5.6k
- Fork
- 511
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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 antirez/sds
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
-
Potential integer overflow in sds.cForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
Issue simili
-
documentation severity: low
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
luainkernel/lunatik#2231 ·
I maintainer di solito rispondono entro 1 giorno
-
Needs-Triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
trezor/trezor-firmware#8210 ·
I maintainer di solito rispondono entro 3 giorni
-
area:storage documentation status:parked
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
nextbsd/nextbsd-userland#519 ·
I maintainer di solito rispondono entro 1 giorno
-
bug system: adam
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
FujiNetWIFI/fujinet-config#358 ·
I maintainer di solito rispondono entro 1 giorno