lib: str_free(NULL) should be a no-op

Updated copyright notices to include the year 2018.

Updated copyright notices to include the year 2017.

global: Change string position/length from unsigned int to size_t Mainly to avoid truncating >4GB strings, which might potentially cause some security holes. Normally there are other limits, which prevent such excessive strings from being created in the first place. I'm sure this didn't find everything. Maybe everything could be found with compiler warnings. -Wconversion kind of does it, but it gives way too many unnecessary warnings. These were mainly found with: grep " = strlen" egrep "unsigned int.*(size|len)"

lib: Optimize str_append_n()

lib: Make str_*() inline which are simple buffer_* wrappers.

global: Avoid using buffer_get_data() Especially buffer_get_data(buf, NULL) generates worse code than just using buf->data directly.

global: Replace buffer_get_used_size(buf) with buf->used Doesn't make any diffence to code generation, but it's somewhat simpler code. It's also more consistent since most of the code nowadays uses the ->used.

global: Use buffer_get_writable_size() where possible With 09539f3db increasing buffer's init_size with +1 some fts-icu unit tests started failing. And in general it's better to use the writable size since that provides the true size that can be used.

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/'

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: pair VA_COPY with va_end A va_copy creates a initialised va_list, as if a va_start had been done on it. Therefore, pedantically, a va_end should also be done on it. On most platforms this is a no-op, and for those where it isn't, the pairing is important. Signed-off-by: Phil Carmody <phil@dovecot.fi>

liblib: Added str_append_data(), which is simply a wrapper to buffer_append() This is intended for appending data that may contain NUL characters that are wanted to be added to the string.

liblib: Fixed crash using str_c() in different stack frame with str_new(0) allocated string.

Updated copyright notices to include year 2014.

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

Updated copyright notices to include year 2013.

buffer: Always keep +1 byte available for str_c()'s NUL.

DEBUG: Call buffer_verify_pool() on str_c*()

Renamed buffer_create_*data() to buffer_create_from_*data() for consistency. Especially i_stream_create_from_data() was a very similar function, which made it difficult to remember which one should have "from" and which one shouldn't.

str_*printfa(): Avoid (harder!) growing underlying buffer if possible. The previous change for this didn't work properly.

str_*printfa(): Avoid growing underlying buffer if possible.

Updated copyright notices to include year 2012.