[Merge] lp:~phablet-team/messaging-framework/set-error-when-remove-self-handle into lp:messaging-framework
Tiago Salem Herrmann
tiago.herrmann at canonical.com
Fri Aug 12 21:42:04 UTC 2016
Review: Needs Fixing
One small remark.
Diff comments:
>
> === modified file 'include/messaging/group_manager.h'
> --- include/messaging/group_manager.h 2016-07-01 13:46:08 +0000
> +++ include/messaging/group_manager.h 2016-08-10 07:04:35 +0000
> @@ -29,6 +29,15 @@
>
> namespace messaging
> {
> +
> +enum class CancelGroupReason
> +{
> + No,
Can we change this to None instead of No? And then update the other MR accordingly. Thanks.
> + Kicked,
> + Gone,
> + Error
> +};
> +
> /// @brief GroupManager models a textual group conversation.
> class MESSAGING_FW_PUBLIC GroupManager : public Interface, NonCopyable, NonMovable
> {
--
https://code.launchpad.net/~phablet-team/messaging-framework/set-error-when-remove-self-handle/+merge/302509
Your team Ubuntu Phablet Team is subscribed to branch lp:messaging-framework.
More information about the Ubuntu-reviews
mailing list