Code review comment for lp:~alecu/unity-scope-click/uninstallable-scopes

Revision history for this message
Paweł Stołowski (stolowski) wrote :

619 +#include <unordered_set>

This doesn't seem to be required in the context of this file?

240 + if (error == click::InterfaceError::NoError) {
241 + uri = "application:///" + val;
242 + }

Can you log an error here, if any?

13 +#include <streambuf>

Is this needed?

review: Needs Fixing

« Back to merge proposal