On St 10. Marec 2010 04:12:13 Matt Rogers wrote:
> On Tuesday 09 March 2010 09:50:40 am Pali Rohár wrote:
> > On Po 8. Marec 2010 20:08:00 you wrote:
> > > On Ne 7. Marec 2010 21:33:44 Raphael Kubo da Costa wrote:
> > > > On Sunday 07 March 2010 15:38:14 Pali Rohár wrote:
> > > > > Hello,
> > > > > 
> > > > > Kopete doesnt set status message after start or after solid
> > > > > networking reconnect.
> > > > > 
> > > > > It is because, setting online status from kopeteapplication or
> > > > > kopeteaccountmanager doesnt contains status message (only empty
> > > > > string).
> > > > > 
> > > > > This patch fix this. I try upload this patch file to reviewboard,
> > > > > but reviewboard show me error: No such revision 1100526 so I
> > > > > sending it here.
> > > > 
> > > > Looks fine overall, even though I don't know this code.
> > > > _______________________________________________
> > > > kopete-devel mailing list
> > > > kopete-devel@kde.org
> > > > https://mail.kde.org/mailman/listinfo/kopete-devel
> > > 
> > > Hello,
> > > 
> > > I see, that status message is not set for account, that comes from
> > > autoaway. It is same as after starting. (only is set global status
> > > message in systray, but for accounts is set empty message).
> > > 
> > > I updated my patch, which set status message after autoaway.
> > 
> > On Ut 9. Marec 2010 04:23:41 Matt Rogers wrote:
> > > On Sunday 07 March 2010 12:38:14 pm Pali Rohár wrote:
> > > > Hello,
> > > > 
> > > > Kopete doesnt set status message after start or after solid
> > > > networking reconnect.
> > > > 
> > > > It is because, setting online status from kopeteapplication or
> > > > kopeteaccountmanager doesnt contains status message (only empty
> > > > string).
> > > > 
> > > > This patch fix this. I try upload this patch file to reviewboard, but
> > > > reviewboard show me error: No such revision 1100526 so I sending it
> > > > here.
> > > 
> > > braces go on separate lines and you're missing braces around the single
> > > line ifs, but otherwise it looks fine and can be committed.
> > 
> > I added missing braces (on separate lines).
> 
> looks fine. please commit.
> 

Commited.
Is this patch for 4.4 branch?

-- 
Pali Rohár
pali.ro...@gmail.com

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel

Reply via email to