Bug #524
Publish only API objects in the installed header files
| Status: | Closed | Start: | ||
| Priority: | Normal | Due date: | ||
| Assigned to: | Andreas Huggel | % Done: | 100% |
|
| Category: | design | |||
| Target version: | 0.18 | |||
Description
It is a basic rule of library design that classes, functions and variables which are only used internally and are not considered part of the library's API should not be present in the installed header files.
They should be moved to new include files, which are not installed.
Consider introducing a new namespace too, e.g., a nested namespace Exiv2::Internal for these objects. Exiv2 cpp files can always start with a 'using namespace Exiv2::Internal' for convenience.
Related issues
| related to Exiv2 - Feature #523 | C++ symbol visibility support for gcc builds | Closed | |||
| related to Exiv2 - Feature #510 | Use pimpl idiom where applicable | New |
History
Updated by Gilles Caulier 569 days ago
Andreas,
We ned to use d private internal class to improve binary compatibilty everywhere. I use this way everywehre in my code (digikam, kipi-plugins and all shared libs relevant)
Other advantage is to have a code which compile faster duing a less of header included everywhere.
If you need help with this task, let's me hear.
Gilles Caulier
Updated by Andreas Huggel 449 days ago
- % Done changed from 0 to 100
Updated by Andreas Huggel 447 days ago
- Status changed from Resolved to Closed