Project

General

Profile

Bug #524

Publish only API objects in the installed header files

Added by Andreas Huggel over 14 years ago. Updated almost 13 years ago.

Status:
Closed
Priority:
Normal
Category:
design
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:

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 buildsClosed

Actions
Related to Exiv2 - Feature #510: Use pimpl idiom where applicableClosed

Actions

History

#1

Updated by Gilles Caulier over 13 years 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

#2

Updated by Andreas Huggel almost 13 years ago

  • % Done changed from 0 to 100
#3

Updated by Andreas Huggel almost 13 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF