jamulussoftware / jamulussoftware/jamulus

Refactor: Improve country code conversion failure modes

Aperta
#2,845 2 commenti 2 reazioni 1 assegnatario Vedi su GitHub

@hoffie ci sta già lavorando.

Dal 9/9/2022.

refactoring
Lingua principale
C
Stelle
1.1k
Fork
248
Merge medio
2g 3h
PR unite (30g)
9

Descrizione

I'd rather "convert to wireformat" and "convert from wireformat" were both "safe" - i.e. returned "Error" (e.g. -1) or a valid response, given any integer value, then have the caller deal with the error as it sees fit. Having to "know" about the IsCountryCodeSupported on top of the conversion methods seems excessively complicated.

Originally posted by @pljones in https://github.com/jamulussoftware/jamulus/pull/2829#discussion_r965319280

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.