From kde-commits Sat Aug 08 07:23:34 2009 From: =?iso-8859-1?q?Andr=E9_W=F6bbeking?= Date: Sat, 08 Aug 2009 07:23:34 +0000 To: kde-commits Subject: Re: KDE/kdebase/workspace/plasma/dataengines/weather/ions Message-Id: <200908080923.34651.Woebbeking () kde ! org> X-MARC-Message: https://marc.info/?l=kde-commits&m=124971625526214 On Friday 07 August 2009, Shawn Starr wrote: > SVN commit 1008478 by spstarr: > > Constify country(). This will always return USA, the ion doesn't > return other countries > > M +1 -1 ion_noaa.cpp > M +1 -1 ion_noaa.h > > > --- > trunk/KDE/kdebase/workspace/plasma/dataengines/weather/ions/ion_noaa > .h #1008477:1008478 @@ -97,7 +97,7 @@ > QMap const& windIcons(void) const; > > // Place information > - QString country(const QString& source) const; > + QString const country(const QString& source) const; The returned QString doesn't need to be const and it's not in the other methods. > QString place(const QString& source) const; > QString station(const QString& source) const; > QString latitude(const QString& source) const;