[Users] Couple of small patches

[wwp]

Hello Olivier,


On Thu, 20 Dec 2018 16:33:21 +0100 Olivier Brunel <jjk at jjacky.com> wrote:

> On Thu, 20 Dec 2018 09:00:04 +0100
> Colin Leroy-Mira <colin at colino.net> wrote:
> 
> > On Wed, 19 Dec 2018 17:12:18 +0100, Olivier Brunel <jjk at jjacky.com>
> > wrote:
> >   
> > >  		procmsg_msginfo_free(&msg);
> > >  
> > > +		gtk_main_iteration_do (FALSE);
> > >  		if (progress_cb != NULL    
> > 
> > Doesn't performing a gtk iteration each message drop search
> > performance badly ?  
> 
> Well, I didn't do extensive testing, but I couldn't really notice any
> diferences when I tried things. Also, I (obviously) have this applied
> here, and - though I'm not doing comparison - my searches don't seem
> slow(er) to me. Of course, it might also depends on other factors,
> one's setup, etc. YMMV.
> 
> (I'd say most of the times GTK should just check that there are no
> pending events and return immediately anyways.)
> 
> So I would say 'no' to the "hurt badly" idea; And though I'm sure it
> does affect it somewhat, for me it doesn't seem noticeable.

I was already doubtful but there you're not really convincing me this
patch could make things better ;-).


Regards,

-- 
wwp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://lists.claws-mail.org/pipermail/users/attachments/20181220/de98c82a/attachment.sig>