qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (QPID-3921) C++ header files need tidyup (in several different ways)
Date Thu, 12 Jun 2014 22:01:02 GMT

    [ https://issues.apache.org/jira/browse/QPID-3921?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14029902#comment-14029902

ASF subversion and git services commented on QPID-3921:

Commit 1602305 from [~astitcher] in branch 'qpid/trunk'
[ https://svn.apache.org/r1602305 ]

QPID-3921: Don't leak use qpid/Options.h header unnecessarily.

> C++ header files need tidyup (in several different ways)
> --------------------------------------------------------
>                 Key: QPID-3921
>                 URL: https://issues.apache.org/jira/browse/QPID-3921
>             Project: Qpid
>          Issue Type: Improvement
>            Reporter: Andrew Stitcher
>            Assignee: Andrew Stitcher
>            Priority: Minor
> There are a number of problems with the header files and their use in the qpid code base:
> 1. There are many header files with non standard header guards -
> - either using incorrect definitions (some starting with double underscore, some with
the header guards in the wrong place in the header file.
> 2. There are many header files that are included unnecessarily, making the compile times
> - This is especially true when a header file is included in another header file instead
of an implementation file.
> - There are (many?) cases where types could be forward declared instead of using the
header file that defines them.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org

View raw message