18
88%
master: 88%

Ran 08 Dec 2016 07:36AM UTC

Files 23

Run time 2s

Badge

Embed ▾

Committed 08 Dec 2016 07:30AM UTC coverage: 76.99% (-0.03%) from 77.024%

Job # LUA="luajit 2.1" ZLIB=lzlib

Build Type

push

travis-ci

Committed by

daurnimator

Commit Message

http/h2_connection: Make sure h2_connection:close() actually closes the connection when called from inside the internal cqueue

The cqueues.poll() calls in h2_connection:close() meant that :close would yield to the current controller. If the current controller was the h2_connection's own cqueue, then nothing would happen until the next action was performed on the h2_connection.
As :close() is usually the *last* thing you do to the h2_connection object, this meant that the socket wouldn't be closed until the garbage collector got around to it.

This fix is to check for a 'close_me' flag upon exit from :step() or :loop() and do the :close() then.
We still need to call cqueues.poll() twice incase other cqueues were waiting on the file descriptor

Run Details

3637 of 4724 relevant lines covered (76.99%)

299.67 hits per line

daurnimator / lua-http / 390 / 18
88%
master: 88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 390.18 (LUA="luajit 2.1" ZLIB=lzlib)

daurnimator / lua-http / 390 / 18 88% master: 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 390.18 (LUA="luajit 2.1" ZLIB=lzlib)

daurnimator / lua-http / 390 / 18
88%
master: 88%

README BADGES
x