Review Request 129400: Avoid potential access of .last() on empty list

David Edmundson david at davidedmundson.co.uk
Tue Nov 22 10:42:40 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129400/
-----------------------------------------------------------

(Updated Nov. 22, 2016, 10:42 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and Plasma.


Repository: kpackage


Description
-------

locateAll can return nothing, therefore we can't always call .last() on
it.

Existing code should be able to handle an empty package root as the user
could also also have provided a broken package root as an argument.


Diffs
-----

  src/kpackagetool/kpackagetool.cpp 6135e14ba717d579553036f9ed7874813046a1db 

Diff: https://git.reviewboard.kde.org/r/129400/diff/


Testing
-------

Ran kpackagetool --generate-index -g

Previously asserted. No longer asserted.


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161122/0aa4e794/attachment.html>


More information about the Kde-frameworks-devel mailing list