[Citadel Development] Re: Merge Request Approval: IMAP memory issues with use of ConstStr

[HarlowSolutions]

While looking at the use of ConstStr, it was interesting how it was being
used.  "Const" is definably not the description to use.  It really is just
a StrBuf where the pointer does not need to be freed.  All sorts of code
change the pointer string and really did not check what it is doing. 
Probably worth replacing the use in our copious spare time lol.