From: Scott C. <can...@os...> - 2007-08-24 04:49:12
|
See diff below. There was one error (strict C++ doesn't permit assignment of non-const iterators to const iterators), one warning I fixed with a using import, and one bug I didn't really know what to do with. CategoryStream is declaring eol twice, once as an enum and once as a function. For the build, I just renamed the enum, but I'll leave the final fix to the maintainers. Once this is all fixed, I'll test OS X. -- Scott cvs diff: Diffing src Index: src/BufferingAppender.cpp =================================================================== RCS file: /cvsroot/log4cpp/log4cpp/src/BufferingAppender.cpp,v retrieving revision 1.1 diff -r1.1 BufferingAppender.cpp 38c38 < for(queue_t::const_reverse_iterator i = queue_.rbegin(), last = queue_.rend(); i != last; ++i) --- > for(queue_t::reverse_iterator i = queue_.rbegin(), last = queue_.rend(); i != last; ++i) cvs diff: Diffing include cvs diff: Diffing include/log4cpp Index: include/log4cpp/CategoryStream.hh =================================================================== RCS file: /cvsroot/log4cpp/log4cpp/include/log4cpp/CategoryStream.hh,v retrieving revision 1.11 diff -r1.11 CategoryStream.hh 40c40 < eol = 0 --- > _eol = 0 Index: include/log4cpp/FixedContextCategory.hh =================================================================== RCS file: /cvsroot/log4cpp/log4cpp/include/log4cpp/FixedContextCategory.hh,v retrieving revision 1.12 diff -r1.12 FixedContextCategory.hh 114a115,116 > using Category::ownsAppender; > |