Re: Review Request: avoid duplicate entries in konqueror address completion list

2012-08-17 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106046/#review17615 --- Ship it! This could be a bit slow, but indeed it sounds necess

Re: Review Request: avoid duplicate entries in konqueror address completion list

2012-08-15 Thread Mark Gaiser
> On Aug. 15, 2012, 7:05 p.m., Mark Gaiser wrote: > > I don't know the konqueror code, but i don't think this is the right way to > > do it. Imho it shouldn't even be inserted in the first place. Though > > someone else would have to confirm. > > Martin Koller wrote: > Well, I see it this

Re: Review Request: avoid duplicate entries in konqueror address completion list

2012-08-15 Thread Martin Koller
> On Aug. 15, 2012, 7:05 p.m., Mark Gaiser wrote: > > I don't know the konqueror code, but i don't think this is the right way to > > do it. Imho it shouldn't even be inserted in the first place. Though > > someone else would have to confirm. Well, I see it this way: there is the list of compl

Re: Review Request: avoid duplicate entries in konqueror address completion list

2012-08-15 Thread Mark Gaiser
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106046/#review17474 --- I don't know the konqueror code, but i don't think this is the

Review Request: avoid duplicate entries in konqueror address completion list

2012-08-15 Thread Martin Koller
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106046/ --- Review request for KDE Base Apps. Description --- When opening e.g. "