D23159: Prevent error() being emitted when purposefully reading 0 bytes

Aleix Pol Gonzalez noreply at phabricator.kde.org
Thu Aug 15 00:20:39 BST 2019


apol added inline comments.

INLINE COMMENTS

> file.cpp:520
> +        // Nothing to do...
> +        return data(QByteArray());
> +    }

this should be

  {
      Q_EMIT data({});`
      return;
  }

I'm not sure that we'd need an empty data emitted then. Are you trying to fix a specific bug that triggers this?

> file.cpp:526
>  
>          if (!res.isEmpty()) {
>              data(res);

alternatively this could also check ` && bytes != 0`

REPOSITORY
  R241 KIO

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

To: feverfew, dfaure, fvogt, chinmoyr
Cc: apol, kfm-devel, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190814/556bbfa2/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list