[Merge] lp:~michael-sheldon/content-hub/fix-content-item-name into lp:content-hub

Ken VanDine ken.vandine at canonical.com
Tue Jul 15 13:58:01 UTC 2014


Review: Needs Information

I see you register the dbus type for cuc::Item in the service's main.cpp, which works fine.  Why not register it in the cucd::Service constructor for places where the service is instantiated outside of main.cpp?  I think the test suite creates a cucd::Service.
-- 
https://code.launchpad.net/~michael-sheldon/content-hub/fix-content-item-name/+merge/226669
Your team Ubuntu Phablet Team is subscribed to branch lp:content-hub.



More information about the Ubuntu-reviews mailing list