davidllewellynjones added inline comments.

INLINE COMMENTS

> ColorTableDestination.cpp:44
> +         handled = false;
>              qCDebug(lcRtf) << "unexpected control word in colortbl:" << 
> controlWord;
>       }

It looks like there may be a distinction to be made between an empty control 
word and an unhandled control word. In the case of an unhandled control word, 
clearing the colour seems sensible, but then in the case of an empty control 
word, the debug output isn't needed.

REPOSITORY
  R8 Calligra

REVISION DETAIL
  https://phabricator.kde.org/D25715

To: pvuorela, davidllewellynjones, dcaliste
Cc: Calligra-Devel-list, davidllewellynjones, dcaliste, ognarb, cochise, 
vandenoever

Reply via email to