Re: Patch: fix the progress information when refreshing an imap folder

[Sergio Villar Senin <svillar igalia com>]

Martin Bonnin escribiu:
> Of course I do this in the application code :)
> 
>> And that's all. With this filter you'll only get progress information
>> about the folder refresh and not about the other operations.
> 
> See the 2 gdb backtraces below. The first one is the"bad" progress.
> The second one is "good". Both have TNY_FOLDER_STATUS_CODE_REFRESH as
> the status code but the first one is called from
> camel_stream_buffer_read_opp() with inconsistent values.

OK, I got your point. This is indeed an issue in tinymail that we need
to fix, and it's the fact that an operation like refresh, provides
different types of information because it's actually a composition of
some others, like opening summary, update summary etc...

Thing is that almost everything that you could do in Camel is like that,
so you'll be only "fixing" particular situations all the time. And
thinking about tinymail POV, as a library in this case, I'm pro offering
as much as info as possible to the apps. Applications should filter it
to present it to the user. You could always use the "what" parameter, I
know it's ugly to compare strings, but I think it's better than fixing
specific situations. What do you think?

Br