libdnspacket
libdnspacket copied to clipboard
`std::optional` instead of `std::tuple`
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 think so