davidllewellynjones accepted this revision. davidllewellynjones added a comment. This revision is now accepted and ready to land.
This looks good to me, and certainly gave better results for me using files with the 'auto' colour. INLINE COMMENTS > pvuorela wrote in ColorTableDestination.cpp:44 > Or well, the code might return such, but think the file in that case is > broken and as such ok to complain about on debug. Also a bit separate from > the changes here. Yeah, this was my misunderstanding of the spec. I don't see a problem with what you're doing here. REPOSITORY R8 Calligra BRANCH rtf_default_color REVISION DETAIL https://phabricator.kde.org/D25715 To: pvuorela, davidllewellynjones, dcaliste Cc: Calligra-Devel-list, davidllewellynjones, dcaliste, ognarb, cochise, vandenoever