On Sat, Aug 05, 2023 at 07:21:02PM +0200, Michiel van den Heuvel wrote:
Do you agree that implementing `IncludeCmd`
would supersede my previous patch?

yeah, i'd say so. it's somewhat more costly at runtime, but that shouldn't matter, and it would be more flexible.

I'll try to cook something up.

cool!

you can probably steal some code from cred_from_cmd() in drv_imap.c.
it's probably not sensible to generalize the existing code, because a) we want to handle multiple lines here (and also accept empty output, i think), and b) the generalization would probably add more code than the de-duplication would save.

regards


_______________________________________________
isync-devel mailing list
isync-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/isync-devel

Reply via email to