Bug 49366

Summary:	mixins should have required base class
Product:	Telepathy	Reporter:	Xavier Claessens <xclaesse>
Component:	tp-glib	Assignee:	Telepathy bugs list <telepathy-bugs>
Status:	RESOLVED FIXED	QA Contact:	Telepathy bugs list <telepathy-bugs>
Severity:	normal
Priority:	medium	CC:	xclaesse
Version:	unspecified	Keywords:	patch
Hardware:	Other
OS:	All
URL:	http://cgit.collabora.com/git/user/xclaesse/telepathy-glib.git/log/?h=next-mixin-base
Whiteboard:
i915 platform:		i915 features:
Bug Depends on:
Bug Blocks:	31668

Description Xavier Claessens 2012-05-02 02:08:48 UTC

I can help to be able to rely on some base class. For example in TpMessageMixin I would like to use tp_base_channel_get_self_handle().


* TpGroupMixin and TpMessageMixin should require to be implemented with TpBaseChannel

* TpPresenceMixin should require to be implemented with TpBaseConnection

* TpPropertiesMixin probably can't require anything

Do we have other mixins?

Comment 1 Jonny Lamb 2012-05-02 02:34:16 UTC

I guess this isn't a problem. I would argue that most of the time you don't need convenience methods from base classes but we should encourage the use of our base classes.

Fine by me.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.