Here's the full patch without printfs and ready for review. There is included some changes in nsImapProtocol.cpp to work around bug 1661992 to avoid an immediate crash so those changes will be removed in later patch versions (function Suspend() and Resume()). Overall, this required a lot of changes to allow folder names to have non-ascii utf-8 chars at all usage points. I think I have checked everything except namespace and server directory name as utf-8. Reporter Alessandro, thanks for letting me use your Courier server to test this. I have also tested it on gmail which also support UTF8=ACCEPT capability. Any testing you could do on this would be appreciated. If that's possible let me know and I will produce a "try" build for you to use. I have manually run my changed version though all the comm unit tests and don't see any failure that can be attributed to my changes.
Bug 1571672 Comment 69 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Here's the full patch without printfs and ready for review. There is included some changes in nsImapProtocol.cpp to work around bug 1661992 to avoid an immediate crash so those changes will be removed in later patch versions (function Suspend() and Resume()). Overall, this required a lot of changes to allow folder names to have non-ascii utf-8 chars at all usage points. I think I have checked everything except namespace and server directory name as utf-8. Reporter Alessandro, thanks for letting me use your Courier server to test this. I have also tested it on gmail which also support UTF8=ACCEPT capability. Any testing you could do on this would be appreciated. If that's possible let me know and I will produce a "try" build for you to use. I have manually run my changed version through all the comm unit tests and don't see any failure that can be attributed to my changes. Finally, clang formatting has yet to be done.