2
38%
master: 38%

Ran 02 Jun 2015 09:15AM UTC

Files 90

Run time 11s

Badge

Embed ▾

Committed 02 Jun 2015 09:13AM UTC coverage: 40.543% (+0.1%) from 40.423%

Job # EXECUTE_TEST_COVERALLS=true

Build Type

Pull #6

travis-ci

Committed by

echampet

Commit Message

Protocol\Smtp->data(): be more memory friendly (strpos/substr version)

With huge mails (50M, 660000 lines), i'm hitting memory_limit, so i've done some tests
looking at peak memory usage and average time(100 runs)
- str_replace("\r","") + explode("\n") 250M/0,27s (what we use now)
- preg_split("\r?\n") 200M/1,5s (too slow)
- strtok("\n") + rtrim("\r") 100M/0,27s (convert multiple \n\n\n in 1 -> ko)
- strpos("\n") + substr() 50M/0,35s
- fwrite(php://temp) + stream_get_line 50M/0,29s

All tests were run with 'php -d"memory_limit=256M" -d"opcache.enable_cli=1"'
With that many iteration (660000), factoring escape+send cost ~0.1s

Before this patch, we were removing \r, now we're keeping them,
in any case \r should be encoded

Signed-off-by: Etienne CHAMPETIER <etienne.champetier@fiducial.net>

Pull Request Pull Request #6: Protocol\Smtp->data(): be more memory friendly (strpos/substr version)

Coverage Stats

1537 of 3791 relevant lines covered (40.54%)

9.45 hits per line

zendframework / zend-mail / 23 / 2
38%
master: 38%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 23.2 (EXECUTE_TEST_COVERALLS=true)

zendframework / zend-mail / 23 / 2 38% master: 38%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 23.2 (EXECUTE_TEST_COVERALLS=true)

zendframework / zend-mail / 23 / 2
38%
master: 38%

README BADGES
x