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

List:       kmail-devel
Subject:    Re: [PATCH] fix bug 43417
From:       Michael =?iso-8859-1?q?H=E4ckel?= <haeckel () kde ! org>
Date:       2002-09-01 8:33:28
[Download RAW message or body]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Friday 30 August 2002 19:04, Carsten Burghardt wrote:
>
> For sure but I need to find out when this listing should be triggered. So
> it should be save to catch the most important errors and to trigger a
> listDirectory. If I don't show a message I'm doing nothing wrong ;-). Or do
> I miss something?

Well, possibly serious performance problems when this is done too often.

To me currently two IMAP commands come into my mind which can give a hint that
possibly the folder does not exist. That are the SELECT and the STATUS
command.
Since the IMAP protocol only knows two error codes:
- - BAD: Command not supported or invalid arguments
- - NO: Command didn't work successfully
the kio slave obviously can't give more detailed information as it gets from
the server.

Ok, we could maybe agree, that the IMAP slave always returns
ERR_CANNOT_ENTER_DIRECTORY if SELECT or STATUS fail and that this error
message is reservered for that, but this should be documented then.

Currently we still always execute also a LIST command before selecting a
folder which also could be used for that, but the IMAP slave should be fixed
to not execute it, because it is not really neccesary.

BTW: Please commit the job->errorText() part of your patch immediately, as 
it's definitely a serious bugfix.

Regards,
Michael Häckel
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)

iD8DBQE9cdDYe9KEPyN2R8URArV1AJ4xOOFRALyTTYaf/3mCNtjeTVnZ8wCfbfeX
Fkf3IaZcAy/oI+MOGPqEZxc=
=8nTx
-----END PGP SIGNATURE-----


_______________________________________________
KMail Developers mailing list
kmail@mail.kde.org
http://mail.kde.org/mailman/listinfo/kmail
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic