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

List:       kde-core-devel
Subject:    Re: multi-url KBookmarkDrag
From:       David Faure <faure () kde ! org>
Date:       2002-03-21 10:15:46
[Download RAW message or body]

On Thursday 21 March 2002 03:46, Alexander Kellett wrote:
> On Thu, Mar 21, 2002 at 12:23:21AM +0100, Alexander Kellett wrote:
> > On Thu, Mar 21, 2002 at 12:15:20AM +0100, David Faure wrote:
> > > Summary: I strongly suggest QValueList<KBookmark>, and committing
> > > before 3.0.
> > 
> > Okay, shall do. I'll post the patch for approval first of course, 
> 
> Okay. Here it is. Tested, works here, but i'd like someone to read over it as
> i've never used qvaluelist before :)
> 
> Haven't had time to test rest of kde* modules but I assume nothing other than
> keditbookmarks depends on KBookmarkDrag::decode().
> 
> It also adds a default ctor to KBookmark as discussed with dfaure on irc, and
> this version lacks the code dup the last one had :)

This looks good to me, I'd say commit.

-- 
David FAURE, david@mandrakesoft.com, faure@kde.org
http://people.mandrakesoft.com/~david/, http://www.konqueror.org/
KDE, Making The Future of Computing Available Today

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