Skip to content

Commit

Permalink
Move compress=false out of packDataDomainNames into caller (#849)
Browse files Browse the repository at this point in the history
This makes packDataDomainNames more consistent with PackDomainName where
it will only be called with compress = true for `dns:"cdomain-name"`.
  • Loading branch information
tmthrgd authored and miekg committed Nov 30, 2018
1 parent 778aa4f commit 6ade5b5
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion msg_generate.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion msg_helpers.go
Original file line number Diff line number Diff line change
Expand Up @@ -624,7 +624,7 @@ func unpackDataDomainNames(msg []byte, off, end int) ([]string, int, error) {
func packDataDomainNames(names []string, msg []byte, off int, compression map[string]int, compress bool) (int, error) {
var err error
for j := 0; j < len(names); j++ {
off, err = PackDomainName(names[j], msg, off, compression, false && compress)
off, err = PackDomainName(names[j], msg, off, compression, compress)
if err != nil {
return len(msg), err
}
Expand Down
2 changes: 1 addition & 1 deletion zmsg.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 6ade5b5

Please sign in to comment.