diff options
author | Corey Lin <514971757@qq.com> | 2018-11-08 19:17:01 +0800 |
---|---|---|
committer | Felix Lange <fjl@users.noreply.github.com> | 2018-11-08 19:17:01 +0800 |
commit | 968f6019d00e349e988f720cdaa065def85ea72f (patch) | |
tree | aecd751429bdc0ba1b3a902657b505ce1af5acbf | |
parent | 503993c819f0a4e7e53f6739e90497106ebdfe09 (diff) | |
download | go-tangerine-968f6019d00e349e988f720cdaa065def85ea72f.tar.gz go-tangerine-968f6019d00e349e988f720cdaa065def85ea72f.tar.zst go-tangerine-968f6019d00e349e988f720cdaa065def85ea72f.zip |
event, event/filter: minor code cleanup (#18061)
-rw-r--r-- | event/event_test.go | 8 | ||||
-rw-r--r-- | event/filter/generic_filter.go | 4 |
2 files changed, 6 insertions, 6 deletions
diff --git a/event/event_test.go b/event/event_test.go index a12945a47..2be357ba2 100644 --- a/event/event_test.go +++ b/event/event_test.go @@ -141,7 +141,7 @@ func TestMuxConcurrent(t *testing.T) { } } -func emptySubscriber(mux *TypeMux, types ...interface{}) { +func emptySubscriber(mux *TypeMux) { s := mux.Subscribe(testEvent(0)) go func() { for range s.Chan() { @@ -182,9 +182,9 @@ func BenchmarkPost1000(b *testing.B) { func BenchmarkPostConcurrent(b *testing.B) { var mux = new(TypeMux) defer mux.Stop() - emptySubscriber(mux, testEvent(0)) - emptySubscriber(mux, testEvent(0)) - emptySubscriber(mux, testEvent(0)) + emptySubscriber(mux) + emptySubscriber(mux) + emptySubscriber(mux) var wg sync.WaitGroup poster := func() { diff --git a/event/filter/generic_filter.go b/event/filter/generic_filter.go index d679b8bfa..467bf01be 100644 --- a/event/filter/generic_filter.go +++ b/event/filter/generic_filter.go @@ -25,7 +25,7 @@ type Generic struct { // self = registered, f = incoming func (self Generic) Compare(f Filter) bool { - var strMatch, dataMatch = true, true + var strMatch = true filter := f.(Generic) if (len(self.Str1) > 0 && filter.Str1 != self.Str1) || @@ -40,7 +40,7 @@ func (self Generic) Compare(f Filter) bool { } } - return strMatch && dataMatch + return strMatch } func (self Generic) Trigger(data interface{}) { |