Son of a ...! It does. I'm now questioning life and my tests.

On Tuesday, December 24, 2013, Matt Reiferson wrote:

> In lib/readerrdy.coffee:
>
> > +      else if @current_state_name in ['TRY_ONE', 'MAX']
> > +        connectionRdy.bump()
> > +
> > +  removeConnection: (conn) ->
> > +    @connections.splice @connections.indexOf(conn), 1
> > +    @roundRobinConnections.remove conn
> > +
> > +    if @connections.length is 0
> > +      @goto 'ZERO'
> > +
> > +  bump: ->
> > +    for conn in @connections
> > +      conn.bump()
> > +
> > +  try: ->
> > +    @roundRobinConnections.next()
>
> doesn't this need to be @roundRobinConnections.next().bump()?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dudleycarr/nsqjs/pull/5/files#r8547020>
> .
>