Merge pull request #182 from clintlombard/fix/empty-polling

Fix Poller poll to handle empty case
pebbe · Feb 18, 2022 · 4333451 · 4333451
2 parents 5806f25 + 7c72941
commit 4333451
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/draft/polling.go b/draft/polling.go
@@ -153,6 +153,10 @@ func (p *Poller) PollAll(timeout time.Duration) ([]Polled, error) {
 func (p *Poller) poll(timeout time.Duration, all bool) ([]Polled, error) {
 	lst := make([]Polled, 0, len(p.items))
 
+	if len(p.items) == 0 {
+		return lst, nil
+	}
+
 	var ctx *Context
 	for _, soc := range p.socks {
 		if !soc.opened {

diff --git a/polling.go b/polling.go
@@ -153,6 +153,10 @@ func (p *Poller) PollAll(timeout time.Duration) ([]Polled, error) {
 func (p *Poller) poll(timeout time.Duration, all bool) ([]Polled, error) {
 	lst := make([]Polled, 0, len(p.items))
 
+	if len(p.items) == 0 {
+		return lst, nil
+	}
+
 	var ctx *Context
 	for _, soc := range p.socks {
 		if !soc.opened {