Feature #510: Use pimpl idiom where applicable - Exiv2

Feature #510

Use pimpl idiom where applicable

Added by Andreas Huggel almost 12 years ago. Updated 3 months ago.

Status:

Closed

Start date:

Priority:

Normal

Due date:

Assignee:

Robin Mills

% Done:

100%

Category:

design

Estimated time:

1.00 hour

Target version:

1.0

Description

Increases stability of the interface
From Sutter, C++ Coding Standards:

class Map {

// ...

private:
struct Impl;
shared_ptr<Impl> pimpl_;
};

Related issues

Associated revisions

Revision 1967
Added by Andreas Huggel about 9 years ago

#579: Implemented mmap for Windows directly in FileIo and made that class use the Pimpl idiom (#510) to de-clutter the interface. Unrelated: Added EXV_MIN/MAX macros, allow iconv config in commercial version.

Revision 1970
Added by Andreas Huggel about 9 years ago

Changed MemIo to use a Pimpl class too (#510).

History

#1 Updated by Andreas Huggel about 10 years ago

Target version set to 1.0

#2 Updated by Robin Mills 3 months ago

Status changed from New to Closed
Assignee set to Robin Mills
% Done changed from 0 to 100
Estimated time set to 1.00

I'm going to close this. Luis wants to rewrite quite a lot of the code in C++11 for Exiv2 v0.28. So this issue is obsolete.

Also available in: Atom PDF

	Related to Exiv2 - Feature #523: C++ symbol visibility support for gcc builds	Closed
	Related to Exiv2 - Bug #524: Publish only API objects in the installed header files	Closed

Exiv2