I will test it tomorrow.
Thanks for your work.
2008/7/24 Johannes Berg johannes@sipsolutions.net:
On Mon, 2008-07-21 at 15:26 +0300, Timo Sirainen wrote:
On Mon, 2008-07-21 at 06:48 +0200, Johannes Berg wrote:
On Sun, 2008-07-20 at 23:53 +0300, Timo Sirainen wrote:
Ok, that seems to work, but I think a better alternative would probably be to make dest_mail a struct mail ** like the context.
That'd be an API change and I'd rather not do that for v1.1. But I suppose it would be the best permanent solution, so I'll do that for v1.2.
Right, yeah, it'd be an API change, though I suppose the only external plugin is probably mine ;) If you wanted to do it you could make some header file declare a macro SAVE_FINISH_HAS_STRUCT_MAIL_PP, but I'm ok with doing it in 1.2, except that means that during 1.1 antispam and quota cannot be used together.
No, this should help with v1.1:
You could then do it like quota plugin and I think it should work.
Ok, I've committed that, so it should work now. Somebody please test.
johannes