common: refactor logic to avoid compiler warnings
cmake: make /WX optional for MSVC
common: avoid calling atobool() on non-boolean strings
NAL: use explicit reset method instead of copy constructor
NAL: cleanup write method

This whole class hierarchy is way over-engineered
NAL: further simplifications
TComBitstream: nits
TEncSbac: cleanup header
TEncSbac: nits
cli: add a line of logging describing reconstructed image file
encoder: use cpu count, rather than pool size, to auto-detect frame threads

Normally the pool size is the same as the detected cpu count, but when WPP is
disabled, the thread count is 1, and this was breaking auto-detection of frame
threads.  Now it will properly use -F4 on desktops when WPP is disabled.