NLnetLabs / simdzone

Fast and standards compliant DNS zone parser
BSD 3-Clause "New" or "Revised" License
63 stars 11 forks source link

Touch-up api documentation and exported "symbols" for 0.1.0 release #180

Closed k0ekk0ek closed 4 months ago

k0ekk0ek commented 4 months ago

Before releasing 0.1.0. Remove defines like ZONE_A and ZONE_IN. The type and class codes are standardized and do not require a define. While defines are sane, my reasoning is that any DNS application will define them and I prefer to avoid collisions. Another thing to do is to properly document the API functions, especially zone_parse_string as it's not obvious to the user that the input must be padded.