I assume that this library is based on C++17 (by using std::variant) — so wouldn't it be better to replace std::tuple with std::optional in the Parse function?
Suggestion:
std::optional<DnsMessage> Parse(const uint8_t *buf, size_t bufSize)
What do you think?
I assume that this library is based on C++17 (by using
std::variant) — so wouldn't it be better to replacestd::tuplewithstd::optionalin theParsefunction?Suggestion:
What do you think?