Redmine#4620: log less on successful link and return correct status on NOOP
Remove several redundant hash functions.
Switch out needless loop with memcmp and kill one variable.
Make file change message "notice" log level. It is not an error.

Redmine #4638.
Memory abuse fix: StackFrame's ->frame may be unset.

Need to initialize it (and assert we're not stomping an old value in
the one place that does set it), as well as free()ing it later.
Leak-fix: free( ->comment) before over-writing it.

It's been set earlier to an xstrdup(), so simply assigning another to
it is negligent.
Fix bogus "return" in void Prefix##SetClear().

The type-specific variant of SetClear was returning SetClear, except
that they're both void.  Many compilers cope with that, but apparently
a user had problems building on Solaris because of it.
Leak-fix: DeletePackageManagers() missed two members.

Need to also delete the patch_list and patch_avail, as well as the
pack_list.
Escaping character magic fails on Windows. See issue #4670
Merge pull request #1453 from aappilattoq/master

Escaping character magic fails on Windows. See issue #4670
Add acceptance test for Redmine #4638.
Merge pull request #1452 from kacfengine/file_changes

Redmine #4638: Cleanups in file changes
Use CfReadFile instead of readfile to avoid confusion with the function
Merge pull request #1454 from ediosyncratic/leak-fixes

Leak fixes
Clean up math eval parser after use
Fix leak in function nth
Fix leak in LaunchParsingMachine (rlist parsing)
Fix leak in function variablesmatching
Fix leak in function regarray
Fix leaks in function makerule