[Merge] lp:~phablet-team/webbrowser-app/webbrowser-app-bookmark-timestamp into lp:webbrowser-app

Olivier Tilloy olivier.tilloy at canonical.com
Tue Jul 29 21:07:11 UTC 2014


> > 226     +        inline bool operator==(BookmarkEntry& a) { return this->url
> > == a.url; }
> >
> > This operator doesn’t seem to be used anywhere. I guess it doesn’t hurt
> much,
> > but it could be removed.
> 
> I added it because it is required by QSet

The QSet used here stores QUrls, not BookmarkEntries.
-- 
https://code.launchpad.net/~phablet-team/webbrowser-app/webbrowser-app-bookmark-timestamp/+merge/224117
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~phablet-team/webbrowser-app/webbrowser-app-bookmark-timestamp into lp:webbrowser-app.



More information about the Ubuntu-reviews mailing list