--Boundary-01=_qJMTN/lA8kijko8 Content-Type: text/plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Em Saturday 05 February 2011, Raphael Kubo da Costa escreveu: > Lamarque Souza writes: > > +bool compareOnlineStatus(const Kopete::Account *a, const Kopete::Account > > *b); +bool invertedCompareOnlineStatus(const Kopete::Account *a, const > > Kopete::Account *b); > > Isn't it better to make them static? I do not know why. They are not declared in any header, nobody can use them outside kopetewindow.cpp. Making them static would not improve anything as far as I know. Anyway, I can do the change. -- Lamarque V. Souza http://www.geographicguide.com/brazil.htm Linux User #57137 - http://counter.li.org/ http://www.kde-mg.org --Boundary-01=_qJMTN/lA8kijko8 Content-Type: text/html; charset="iso-8859-15" Content-Transfer-Encoding: 7bit

Em Saturday 05 February 2011, Raphael Kubo da Costa escreveu:

> Lamarque Souza <lamarque@gmail.com> writes:

> > +bool compareOnlineStatus(const Kopete::Account *a, const Kopete::Account

> > *b); +bool invertedCompareOnlineStatus(const Kopete::Account *a, const

> > Kopete::Account *b);

>

> Isn't it better to make them static?


I do not know why. They are not declared in any header, nobody can use them outside kopetewindow.cpp. Making them static would not improve anything as far as I know. Anyway, I can do the change.


--

Lamarque V. Souza

http://www.geographicguide.com/brazil.htm

Linux User #57137 - http://counter.li.org/

http://www.kde-mg.org

--Boundary-01=_qJMTN/lA8kijko8--