From 36808cdec7768c4d9744f126c3d6a078633ae3ad Mon Sep 17 00:00:00 2001 From: Alison Winters Date: Fri, 1 Nov 2019 23:27:53 +0800 Subject: [PATCH] remove unused patternType return --- dnscrypt-proxy/pattern_matcher.go | 12 ++++++------ dnscrypt-proxy/plugin_block_name.go | 2 +- dnscrypt-proxy/plugin_whitelist_name.go | 2 +- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/dnscrypt-proxy/pattern_matcher.go b/dnscrypt-proxy/pattern_matcher.go index 17f1606b..00117546 100644 --- a/dnscrypt-proxy/pattern_matcher.go +++ b/dnscrypt-proxy/pattern_matcher.go @@ -51,7 +51,7 @@ func isGlobCandidate(str string) bool { return false } -func (patternMatcher *PatternMatcher) Add(pattern string, val interface{}, position int) (PatternType, error) { +func (patternMatcher *PatternMatcher) Add(pattern string, val interface{}, position int) error { leadingStar := strings.HasPrefix(pattern, "*") trailingStar := strings.HasSuffix(pattern, "*") exact := strings.HasPrefix(pattern, "=") @@ -60,24 +60,24 @@ func (patternMatcher *PatternMatcher) Add(pattern string, val interface{}, posit patternType = PatternTypePattern _, err := filepath.Match(pattern, "example.com") if len(pattern) < 2 || err != nil { - return patternType, fmt.Errorf("Syntax error in block rules at pattern %d", position) + return fmt.Errorf("Syntax error in block rules at pattern %d", position) } } else if leadingStar && trailingStar { patternType = PatternTypeSubstring if len(pattern) < 3 { - return patternType, fmt.Errorf("Syntax error in block rules at pattern %d", position) + return fmt.Errorf("Syntax error in block rules at pattern %d", position) } pattern = pattern[1 : len(pattern)-1] } else if trailingStar { patternType = PatternTypePrefix if len(pattern) < 2 { - return patternType, fmt.Errorf("Syntax error in block rules at pattern %d", position) + return fmt.Errorf("Syntax error in block rules at pattern %d", position) } pattern = pattern[:len(pattern)-1] } else if exact { patternType = PatternTypeExact if len(pattern) < 2 { - return patternType, fmt.Errorf("Syntax error in block rules at pattern %d", position) + return fmt.Errorf("Syntax error in block rules at pattern %d", position) } pattern = pattern[1:] } else { @@ -112,7 +112,7 @@ func (patternMatcher *PatternMatcher) Add(pattern string, val interface{}, posit default: dlog.Fatal("Unexpected block type") } - return patternType, nil + return nil } func (patternMatcher *PatternMatcher) Eval(qName string) (reject bool, reason string, val interface{}) { diff --git a/dnscrypt-proxy/plugin_block_name.go b/dnscrypt-proxy/plugin_block_name.go index 1d6785c8..82b2c707 100644 --- a/dnscrypt-proxy/plugin_block_name.go +++ b/dnscrypt-proxy/plugin_block_name.go @@ -59,7 +59,7 @@ func (plugin *PluginBlockName) Init(proxy *Proxy) error { weeklyRanges = &weeklyRangesX } } - if _, err := plugin.patternMatcher.Add(line, weeklyRanges, lineNo+1); err != nil { + if err := plugin.patternMatcher.Add(line, weeklyRanges, lineNo+1); err != nil { dlog.Error(err) continue } diff --git a/dnscrypt-proxy/plugin_whitelist_name.go b/dnscrypt-proxy/plugin_whitelist_name.go index 6af856ec..80cce74c 100644 --- a/dnscrypt-proxy/plugin_whitelist_name.go +++ b/dnscrypt-proxy/plugin_whitelist_name.go @@ -59,7 +59,7 @@ func (plugin *PluginWhitelistName) Init(proxy *Proxy) error { weeklyRanges = &weeklyRangesX } } - if _, err := plugin.patternMatcher.Add(line, weeklyRanges, lineNo+1); err != nil { + if err := plugin.patternMatcher.Add(line, weeklyRanges, lineNo+1); err != nil { dlog.Error(err) continue }