Merge pull request #1377 from tzz/doc/changelog-edits

ChangeLog edits and TODOs
Fixing issues with freeing regex.
pcre_free() used instead of ordinary free().
Redmine#2038: adding string_split function to replace splitstring

Deprecate splitstring()

Add StringMatchWithPrecompiledRegex() and CompileRegex() to avoid
compiling regex multiple times.

Fix compilation warning about const Rlist.
Merge pull request #1378 from tzz/tzz/pasinskim/feature/split_string

Redmine#2038: add string_split function
ChangeLog: note deprecations
readfile() acceptance test with large files
Redmine#4217: make readfile() and CfReadFile() less verbose
Merge pull request #1380 from tzz/bugfix/redmine4217-quieten-readfile

Redmine#4217: make readfile() and CfReadFile() less verbose
Fix signature for FnCallSplitString

Close #4222
FileSantityChecks does not need EvalContext
Ensure lists are not saved with non-scalars in verify_vars

Hopefully now Close #4222
tests/acceptance/01_vars/02_functions/variable_references_from_function_that_never_succeeds.cf
ChangeLog: one new feature was actually a bug fix
Redmine#4220: test for 'nothing to repeat' PCRE error
Redmine#4219: catch services 'cannot be resolved' error
Don't use Attribute-laden ItemList save/load functions

/etc/fstab and .log files are not under edit_file editing
model, so their load/save should not be influenced by Attributes
struct.
Move RawSaveItemList to the module it is declared in
Fix return type and argument name for accurateness
if (foo) {} is not a very useful construct
Don't rely on combining variables from several compilation units