Bug 49366 - mixins should have required base class
Summary: mixins should have required base class
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-glib (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL: http://cgit.collabora.com/git/user/xc...
Whiteboard:
Keywords: patch
Depends on:
Blocks: tp-glib-1.0
  Show dependency tree / graph
 
Reported: 2012-05-02 02:08 UTC by Xavier Claessens
Modified: 2012-06-29 03:31 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:

Attachments

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.