The Proton C code has a lot of redundant includes.  That is, headers often
directly include items they already have transitively.  This makes some of
the dependency graphs in our api doc less attractive and less helpful:


http://qpid.apache.org/releases/qpid-proton-0.8/protocol-engine/c/api/connection_8h.html

http://qpid.apache.org/releases/qpid-proton-0.8/protocol-engine/c/api/message_8h.html

http://qpid.apache.org/releases/qpid-proton-0.8/protocol-engine/c/api/messenger_8h.html
(this is a really good one)

http://qpid.apache.org/releases/qpid-proton-0.8/protocol-engine/c/api/transport_8h.html

If there's no technical objection, I'd like to produce a patch to remove
the unnecessary includes.  Are there any platform or compiler issues I
should be aware of?

Justin

Reply via email to