On Thu, Dec 13, 2012 at 02:27:13PM -0800, Andrew Gaul wrote:
I prefer the forward declaration; requiring a specific #include ordering makes code brittle. In this case, we would have to swap imap_private.h
I agree the specific ordering makes it brittle, but so does the duplicate type declaration. I'd rather make imap_privatge.h include message.h if it requires it.