D18144: Implemented find function from console

Albert Astals Cid noreply at phabricator.kde.org
Wed Feb 6 21:15:35 GMT 2019


aacid added a comment.


  Do you think you could create an autotest for this? If not it's fine, but would make sure things don't break in the future.

INLINE COMMENTS

> shell.cpp:684
>      KDocumentViewer* const doc = qobject_cast<KDocumentViewer*>(part);
> +    const QString find(QByteArray::fromBase64(ShellUtils::find(serializedOptions).toLatin1()));
>      if ( ShellUtils::startInPresentation(serializedOptions) )

the fromBase64 should be in ShellUtils itself, the serialization/unserialization is what that "class" does, should not leak outside.

REPOSITORY
  R223 Okular

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

To: joaonetto, #okular
Cc: ngraham, aacid, yurchor, okular-devel, tfella, darcyshen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20190206/a6c955be/attachment-0001.html>


More information about the Okular-devel mailing list