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

List:       kde-pim
Subject:    Re: [Kde-pim] akregator patch
From:       Frank Osterfeld <frank.osterfeld () gmx ! de>
Date:       2007-11-18 10:56:30
Message-ID: 200711181156.31534.frank.osterfeld () gmx ! de
[Download RAW message or body]

On Sunday 18 November 2007 11:17:46 Kleag wrote:
> Sorry, my patch was probably false. To define a real order relation, it
> should be like that, I think:
> Index: akregator/src/articlejobs.h
> ===================================================================
> --- akregator/src/articlejobs.h (révision 738190)
> +++ akregator/src/articlejobs.h (copie de travail)
> @@ -42,7 +42,8 @@
>      QString guid;
>      bool operator<( const ArticleId& other ) const
>      {
> -        return feedUrl < other.feedUrl && guid < other.guid;
> +        return ( (feedUrl < other.feedUrl)
> +                || (feedUrl == other.feedUrl && guid < other.guid));
>      }
>  };
>

That looks correct to me (at least on a sunday morning :) ).

> > Can I apply it ?

Yes, go ahead.

Thanks,

Frank
_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/

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