Feature #510

Use pimpl idiom where applicable

Added by Andreas Huggel over 11 years ago. Updated 25 days ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Robin Mills% Done:

100%

Category:designEstimated 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

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

Associated revisions

Revision 1967
Added by Andreas Huggel almost 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 almost 9 years ago

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

History

#1 Updated by Andreas Huggel almost 10 years ago

  • Target version set to 1.0

#2 Updated by Robin Mills 25 days 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

Redmine Appliance - Powered by TurnKey Linux