Options cleanup - Githubissues

bifurcation / mint

A Minimal TLS 1.3 Implementation in Go

MIT License

225 stars 36 forks source link

Options cleanup #213

Closed bifurcation closed 4 years ago

bifurcation commented 4 years ago

Depends on #212

This PR cleans up a few things about how struct tags are handled:

Instead of parsing into a generic map[string]int, we now use a struct
This struct is used in place of encOpts and decOpts, since they're the same anyway
Tags are now checked for consistency at parse time
Slice encoding is now a bit more systematic