Unify dns_*() function definitions for all platforms
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C
- Sterne
- 40.4k
- Forks
- 8.2k
- Ø Merge
- 2 T. 13 Std.
- Gemergte PRs (30 T.)
- 96
Beschreibung
Description
This is a todo in the code
I'm putting it here so it gets on the radar for a todo.
If this is a wontfix then remove the todo from the code.
Changes to this code segment caused an issue #7748 with the output of gethostbyaddr() showing as a binary string in Windows.
/* TODO: Reimplement the gethostby* functions using the new winxp+ API, in dns_win32.c, then
we can have a dns.c, dns_unix.c and dns_win32.c instead of a messy dns.c full of #ifdef
*/
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem TODO in ext/standard/dns.c etwa bei Zeile 138 und vergleiche anschließend die Organisation der referenzierten Dateien dns.c, dns_unix.c und dns_win32.c. Prüfe Issue #7748 auf die Windows-Regression bei gethostbyaddr(). Als abgeschlossen gilt die Aufgabe, wenn entweder die Definitionen von dns_*() plattformübergreifend ohne diese Regression vereinheitlicht wurden oder das TODO entfernt wird, falls das Projekt entscheidet, dass es ein wontfix ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- c
- Bereich
- networking
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100