[Kde-pim] akregator patch

Frank Osterfeld frank.osterfeld at gmx.de
Sun Nov 18 10:56:30 GMT 2007


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 at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/


More information about the kde-pim mailing list