[Citadel Development] (no subject)

[dothebart]

 

Thu Jul 11 2013 06:35:09 EDT from dothebart @ Uncensored

 

Wed Jul 10 2013 21:09:27 EDT from IGnatius T Foobar @ Uncensored

Ok, fixed. StrBufStripAllBut() now returns the leftmost qualifying substring.

did you check whether there are other places which might not like that behaviour?

8.1x-next now?

(cherry pick the last two commits over - webcit vcard fix plus yours)

                StrBufStripAllBut(sSMTP->from, '<', '>');

        StrBufStripAllBut(sSMTP->OneRcpt, '<', '>');
./citadel/modules/smtp/serv_smtp.c
                StrBufStripAllBut(section, '[', ']');
                StrBufStripAllBut(partial, '<', '>');
./citadel/modules/imap/imap_fetch.c

I still don't think thats a clever solution to change the methods way of working - i'd rather add a new function with another name for this.