address issue #136
enable renderReq only after delayRC has been fired
Vladimir Bauer
2 years ago
| 67 | 67 | ctx = context.Background() |
| 68 | 68 | } |
| 69 | 69 | ctx, cancel := context.WithCancel(ctx) |
| 70 | delayRC := make(chan struct{}, 1) | |
| 71 | delayRC <- struct{}{} | |
| 70 | 72 | s := &pState{ |
| 71 | 73 | ctx: ctx, |
| 72 | 74 | hm: make(heapManager), |
| 73 | 75 | dropS: make(chan struct{}), |
| 74 | 76 | dropD: make(chan struct{}), |
| 75 | 77 | renderReq: make(chan time.Time), |
| 78 | popPriority: math.MinInt32, | |
| 76 | 79 | refreshRate: defaultRefreshRate, |
| 77 | popPriority: math.MinInt32, | |
| 80 | delayRC: delayRC, | |
| 78 | 81 | queueBars: make(map[*Bar]*Bar), |
| 79 | 82 | output: os.Stdout, |
| 80 | 83 | debugOut: io.Discard, |
| 259 | 262 | defer p.pwg.Done() |
| 260 | 263 | render := func() error { return s.render(cw) } |
| 261 | 264 | var err error |
| 265 | var renderReq <-chan time.Time | |
| 262 | 266 | |
| 263 | 267 | for { |
| 264 | 268 | select { |
| 266 | 270 | op(s) |
| 267 | 271 | case fn := <-p.interceptIO: |
| 268 | 272 | fn(cw) |
| 269 | case <-s.renderReq: | |
| 273 | case <-s.delayRC: | |
| 274 | renderReq = s.renderReq | |
| 275 | s.delayRC = nil | |
| 276 | case <-renderReq: | |
| 270 | 277 | e := render() |
| 271 | 278 | if e != nil { |
| 272 | 279 | p.cancel() // cancel all bars |
| 293 | 300 | } |
| 294 | 301 | |
| 295 | 302 | func (s pState) autoRefreshListener(done chan struct{}) { |
| 296 | if s.delayRC != nil { | |
| 297 | <-s.delayRC | |
| 298 | } | |
| 299 | 303 | ticker := time.NewTicker(s.refreshRate) |
| 300 | 304 | defer ticker.Stop() |
| 301 | 305 | for { |