Feature #510
Use pimpl idiom where applicable
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
Associated revisions
Changed MemIo to use a Pimpl class too (#510).
History
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.
#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.