D28479: use httpAssembleURI not httpAssembleURIf
Harald Sitter
noreply at phabricator.kde.org
Wed Apr 1 11:37:10 BST 2020
sitter created this revision.
sitter added a reviewer: dantti.
Herald added a subscriber: kde-utils-devel.
sitter requested review of this revision.
REVISION SUMMARY
the latter is for when the last argument is a format string. which I
believe is not the case here. or perhaps more importantly if it was a
format string it'd be an invalid call because we pass no values in
TEST PLAN
- make two printers
- pause them
- print job to one printer
- open printer queue
- right click job to move to other printer
- job is now on other printer
REPOSITORY
R363 Print Manager
BRANCH
release/20.04
REVISION DETAIL
https://phabricator.kde.org/D28479
AFFECTED FILES
libkcups/KIppRequest.cpp
To: sitter, dantti
Cc: kde-utils-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20200401/07389da2/attachment.html>
More information about the Kde-utils-devel
mailing list