Re: [PATCH] "Reply" button functionality in KMail (especially with mailing lists)

On Sunday 25 November 2007, Allen Winter wrote:
> On Saturday 24 November 2007 14:05:32 Matthew Blissett wrote:
> > Hi
> >
> > (Some deleted quotation replaced)
> >
> > I've made a patch!  It's my first one, so let me know what can be
> > improved. It applies to kdepim/kmail/kmmessage.cpp
>
> The patch applies cleanly to trunk.
> The code looks good and you even followed our coding style!
> The code compiles too :)
>
> So I'll start using it locally.   And if I don't find any problems
> within the next few days I will commit.  Unless there are objections.

FYI: I object. For details see my reply to Matthew's message.


Regards,
Ingo

_______________________________________________
KMail developers mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kmail-devel

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.4-svn0 (GNU/Linux)

iD8DBQBHTKgrGnR+RTDgudgRAtpjAKC9F3wJqxXMuPpmeMgXPfeFbd9bOgCZATWR
2hMOztjYUSkZ/IKEBdNATC8=
=2TMw
-----END PGP SIGNATURE-----