fvogt added inline comments.

INLINE COMMENTS

> notificationsengine.cpp:266
>      const int WORD_PER_MINUTE = 250;
> -    int count = summary.length() + body.length();
> +    int count = summary.length() + (body.length() - 33); // 33 is the length 
> of <?xml> and <html> tags
>  

I don't like magic numbers, maybe `sizeof("<?xml 
version="1.0"><html></html>")-1`?

REPOSITORY
  R120 Plasma Workspace

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

To: broulik, #plasma, davidedmundson, aacid, fvogt
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart

Reply via email to