Project

General

Profile

Feature #510

Use pimpl idiom where applicable

Added by Andreas Huggel over 14 years ago. Updated about 3 years ago.

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

100%

Estimated time:
1.00 h

Description

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

class Map {

// ...

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


Related issues

Related to Exiv2 - Feature #523: C++ symbol visibility support for gcc buildsClosed

Actions
Related to Exiv2 - Bug #524: Publish only API objects in the installed header filesClosed

Actions

Associated revisions

Revision 1967 (diff)
Added by Andreas Huggel almost 12 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 (diff)
Added by Andreas Huggel almost 12 years ago

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

History

#1

Updated by Andreas Huggel almost 13 years ago

  • Target version set to 1.0
#2

Updated by Robin Mills about 3 years ago

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

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