[Telepathy] [PATCH] Implement the != operator for Tp::Presence and Tp::PresenceSpec
Rohan Garg
rohan16garg at gmail.com
Fri Sep 30 06:24:49 PDT 2011
Hi Andre
Thanks! This is the first time I'm submitting a patch to Telepathy Qt which
is why i did not know the correct procedure. I'll keep this in mind when
submitting my next set of patches.
Regards
Rohan Garg
On Fri, Sep 30, 2011 at 6:20 AM, Andre Moreira Magalhaes <
andre.magalhaes at collabora.co.uk> wrote:
> Hey Rohan,
>
> On Thu, 2011-09-29 at 00:12 +0530, Rohan Garg wrote:
>
> > I was working on Telepathy KDE and i noticed that the '!=' operator
> > had not been implemented in Tp::Presence and Tp::PresenceSpec. This
> > patch fixes that. I know that one could easily use the '==' operator
> > and implement this functionality in the application using Tp::Presence
> > itself, but i think that this should be implemented by the
> > Tp::Presence class itself in order to keep code using the Telepathy
> > Qt4 API cleaner.
> First thanks for the patch. The patch is fine and I will apply it to
> upstream master to be released in 0.7.3. As for future patches, I would
> like to ask you to please file a bug against Telepathy/tp-qt4 on
> bugs.freedesktop.org, with the patch keyword, so it's easier to keep
> track and make review comments.
>
> BR,
> Andre
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/telepathy/attachments/20110930/d779f344/attachment.htm>
More information about the telepathy
mailing list