cfeck added inline comments.

INLINE COMMENTS

> jtamate wrote in rgb.cpp:702
> Shouldn't it be QLatin1String("\x01\xda\x01")?
> startsWith has a QLatin1String overloaded method, but will it be used if a 
> char* is used as argument or will it use the QString method? I just don't 
> know the answer.
> Otherwise, +1.

`head` is a QByteArray, not a QString.

REPOSITORY
  R287 KImageFormats

BRANCH
  master

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

To: dfaure, cfeck
Cc: jtamate, kde-frameworks-devel, michaelh, ngraham, bruns

Reply via email to