GitLab

Since LogBuffer no longer exists, the file name did not match its content anymore

and pull its functionality into BufferAppender
(one unit test temporarily disabled, might be removed in next commit or updated to get around the protected method it uses)

LogBuffers are now members of BufferAppender instances, and the singleton has been removed. LoggerManagerImpl keeps them (if any) around while processing a new specification, and flushes them when done.

- update README
- copy vector of event on flush
- only use very basic functions in case of flush-on-destruct

To prevent accidental outside use, all LogBugger (and BufferAppender) code is in the namespace isc::log::internal now, and the files have been renamed to _impl

And clean up the process() calls and processSpecification() method (which should mostly look like they were now)

slightly changed outer layer process() call so it isn't necessary anymore

including docstring updates

Also updated log's README

needs cleanup and tests

- reduced a line that was 1 character too long
- flipped order of check with isdigit()
- renamed Token::NUMBER_RANGE to Token::NUMBER_OUT_OF_RANGE

If omitted the build fails on Solaris complaining that distance is unknown.

Which got accidentally removed in the previous commit