--===============81149123032972403==
Content-Type: multipart/signed;
  protocol="application/pgp-signature";
  micalg=pgp-sha1;
  boundary="Boundary-02=_QhFJ/5VmhHray7K";
  charset="iso-8859-1"
Content-Transfer-Encoding: 7bit


--Boundary-02=_QhFJ/5VmhHray7K
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

On Sunday 27 July 2003 00:16, Andreas Gungl wrote:
> Hi,
>
> as discussed long enough on this list, this time I tried to use an
> approach to separate only existing core and UI code. Those who
> reviewed my last patch will have it easy to compare to this one. I
> used kmsender as an example again. And again two issues are open (the
> interactive messagebox and the password dialog). Both require
> significantly more work, so before I continue I better ask about your
> opinions now.
> Special hotspots are:
> - Is my choosen way of handling the strings for the UI okay?
> - I used direct calls to the KMClientInterface's methods. I believe
> that this is appropriate for the moment. If necessary we can switch
> to signals/slots later.

Are those calls necessary at all? Wouldn't it be much better, i. e.=20
cleaner and easier to implement, if the core methods would simply=20
return an error code. And then the UI code would display an appropriate=20
message box.
The progress info probably has to be implemented with back-calls. But=20
all those call-backs just to display an error message might be=20
overkill.

The two open issues could maybe also easily be solved by returning an=20
error code and letting the UI code ask for the user's decision or the=20
password.

Regards,
Ingo


--Boundary-02=_QhFJ/5VmhHray7K
Content-Type: application/pgp-signature
Content-Description: signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3rc1 (GNU/Linux)

iD8DBQA/JFhQGnR+RTDgudgRApVVAKCH/2a0vflkmlqZju8HnrR4p/yG+QCgow3B
yllDpHR4eQa/bMYVMJZbxIA=
=qq+V
-----END PGP SIGNATURE-----

--Boundary-02=_QhFJ/5VmhHray7K--

--===============81149123032972403==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
KMail Developers mailing list
kmail@mail.kde.org
http://mail.kde.org/mailman/listinfo/kmail

--===============81149123032972403==--