Updated copyright notices to include the year 2018.

global: Avoid i_stream_is_eof() where it's not necessary

global: use i_rand_limit() and i_rand_minmax() when possible

global: Replace rand with i_rand

lib: Fix memory leak in test-istream-concat

lib: Fix crash when seeking istream-concat to EOF and trying to read it

Updated copyright notices to include the year 2017.

Remove t_malloc in favour of t_malloc_no0 Using either t_malloc_no0 or t_malloc0 makes it clear whether the allocated memory is zeroed or not.

lib: istream-concat now seeks parent streams to correct offset. All of the streams' offsets were somewhat random.

global: freshen copyright git ls-files | xargs perl -p -i -e 's/(\d+)-201[0-5]/$1-2016/g;s/ (201[0-5]) Dovecot/ $1-2016 Dovecot/'

Remove now-unnecessary direct stdlib.h #includes.

global: freshen copyright Robomatically: git ls-files | xargs perl -p -i -e 's/(\d+)-201[0-4]/$1-2015/g;s/ (201[0-4]) Dovecot/ $1-2015 Dovecot/' Happy 2015 everyone! Signed-off-by: Phil Carmody <phil@dovecot.fi>

lib: test-istream-concat - clean up our test-istreams This reduces noise when memory leak debugging the unit tests. Signed-off-by: Phil Carmody <phil@dovecot.fi>

lib: test-istream-concat - add simultanious limit streams back into the mix Return to the previous complex nested stream-type case. 1 time in 10, it tests the simple case of: test_stream \ test_stream -}- concat_stream test_stream / (2-11 of these) 9 times in 10, it tests this configuration instead: test_stream \ / limit_stream test_stream -}- concat_stream {- limit_stream test_stream / \ limit_stream (2-11 of these) (1-9 of these) Since 31efe2d04793 lib: istream-concat read() returned -2 too early., all tests pass every time. Signed-off-by: Phil Carmody <phil@dovecot.fi>

lib: test-istream-concat - test only concat, not simultanious limit streams Test just concat functionality in this unit test. Simultanious access of limit streams can be tested elsewhere. Without the fix in: 31efe2d04793 lib: istream-concat read() returned -2 too early. The failure previously seen in test-istream-concat would be still reproducable: test-istream-concat.c:84: Assert failed: size >= TEST_MAX_BUFFER_SIZE istream concat random ................................................ : FAILED test: random seed #1 was 1403118493 Signed-off-by: Phil Carmody <phil@dovecot.fi>

lib: remove unwanted srand()s from unit tests We'll get better coverage without them. Note: this change causes the following test case failure occasionally: test-istream-concat.c:88: Assert failed: size >= TEST_MAX_BUFFER_SIZE istream concat random ................................................ : FAILED test: random seed #1 was 1403027537 (The seed may vary, obviously.) Signed-off-by: Phil Carmody <phil@dovecot.fi>

lib: test-istream-concat - early abort test loop on failure No point in continuing past the first failure. Signed-off-by: Phil Carmody <phil@dovecot.fi>

Updated copyright notices to include year 2014.

test-istream-concat unit test improved. Still pretty ugly to test with randomness, but at least it caught the bug in v2.1.

Oops :) Update copyrights to 2013 without breaking all .c files.

Updated copyright notices to include year 2013.

Added i_stream_get_data_size(). Used it where possible.

Updated copyright notices to include year 2012.