keepsimple1 / mdns-sd

Rust library for mDNS based Service Discovery
Apache License 2.0
96 stars 37 forks source link

feat: replace remaining box dns with declared type #206

Closed CosminPerRam closed 5 months ago

CosminPerRam commented 5 months ago

After skimming through the code a bit, I found two usages of Box<dyn DnsRecordExt> instead of using the DnsRecordBox type (Box<dyn DnsRecordExt + Send>). Although in the replaced remaining cases, we dont really use the Send bound but this can be done for consistency sake as there are already other usages of it in places where it isn't necessarily needed.