[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: KDE/kdebase/workspace/plasma/dataengines/weather/ions
From:       André_Wöbbeking <Woebbeking () kde ! org>
Date:       2009-08-08 7:23:34
Message-ID: 200908080923.34651.Woebbeking () kde ! org
[Download RAW message or body]

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<QString, WindDirections> 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;

[prev in list] [next in list] [prev in thread] [next in thread]