- Fix compilation errors, missing dllexport and warnings on
Windows and/or MSVC (Daniela Engert, Marcel Raad)
- Fix unintialized fields (Zoey Greer)
- Stop options with implicit value from consuming separate tokens (Michael John Decker)
- Make multitoken limit be max int, not 32K (Hans Hohenfeld)
- Code formatting and documentation fixes (Jurko, Lauri Nurmi)
- Minimal support for no-rtti build (Minmin Gong)
- Don't increment environment pointer past the end (Vladimir Prus)
Value-initialize m_desc (to NULL) in this constructor to avoid uninitialized memory situations. This issue is related to Coverity CID12523 and was uncovered by Coverity.
m_implicit was previously uninitialized in this constructor, leading to possibly inconsistent functionality. Instead, we now initialize it to false. This issue is related to Coverity CID10559 and was uncovered by Coverity.
Documentation cleanups from Jurko Gospodnetić:
- trim trailing spaces
- fix documentation typos & style
- escape first dash ('-') in double-dashes ('--') in doxygen comments
This should prevent doxygen from generating bad documentation HTML pages
containing '<ndash></ndash>' where those double-dashes should be.
typo corrections:
- otherwisee --> otherwise
- it's is --> it is
- nuber --> number
- will likely to use --> is likely to use
- varaible --> variable
- defauled --> defaulted
stylistic changes:
- which would make --> making
This should not effect correct programs, since once the
iterator itself is past-the-end, it should not matter what
the underlying state of that iterator is. But, the new behaviour
is more obvious.
Even when compiled at warning level 4 (i.e. all), project
policies may require compilations without warnings issued.
Signed-off-by: Daniela Engert <dani@ngrt.de>
Previous fix 4ae33c for ticket #6797 introduced a new problem with
Visual Studio compilers about two missing methods in
'common_config_file_iterator'. Adding those seemingly not required
methods as empty methods fixes the issue. As an added bonus, the
bogus warnings about DLL-interfaces get a silencing treatment, too.
Tests were run with vc10, vc11, and vc12. Without this fix, the
affected test cases fail on all 3 compilers. With the fix in place,
all tests pass.
Signed-off-by: Daniela Engert <dani@ngrt.de>
