Issue #6 - refactored BeanReaderInstructions to only hold values, not to create the bean instance, which is now delegated back to the BeanReaderImpl
Issue #5 - the header can now be read directly from CsvReader. Refactored the underlying mechanism to act as a lazy header reader
Issue #9 - the logging of the settings is now postponed until the moment the Reader actually starts to get read
Issue #1 - basic functionality in place to ignore comment lines
Issue #2 - Added test for empty lines
Note quite happy with the changes to LineReaderImplementation. Next mission is to strip the line number out of LineReader and give it to the state machine
Solved issues #11, #12, #13 and #14 by encapsulating the line skipper and line maintenance in the state machine. Code looks a lot better, less smells