diff options
author | Jay Berkenbilt <ejb@ql.org> | 2019-08-31 00:35:45 +0200 |
---|---|---|
committer | Jay Berkenbilt <ejb@ql.org> | 2019-08-31 00:36:44 +0200 |
commit | 00fd9b3e035c9c155f15a52ef7dfa3366d35662f (patch) | |
tree | 8e95abb8f6d89293f8c63e0cf6b5d25d49f81cfb /TODO | |
parent | bde44c18ef3d6302dae723e6ef431f87e5764ec9 (diff) | |
download | qpdf-00fd9b3e035c9c155f15a52ef7dfa3366d35662f.tar.zst |
Update C++-11 notes
Diffstat (limited to 'TODO')
-rw-r--r-- | TODO | 22 |
1 files changed, 13 insertions, 9 deletions
@@ -15,15 +15,19 @@ C++-11 (for qpdf 10) * My c++11 branch adds autoconf tests to require C++-11 and re-implements PointerHolder so that it is interchangeable with - std::shared_ptr. Once this is in master, it will be possible to - globally replace PointerHolder with std::shared_ptr. This will break - binary compatibility, but if people want source compatibility, they - just have to explicitly include qpdf/PointerHolder.hh. We should - actually look at every use of PointerHolder to see which ones need - to be std::shared_ptr and which ones can be std::unique_ptr. For - source compatibility, all uses of PointerHolder in the API should be - changed to std::shared_ptr, though most likely they have to be that - anyway since std::unique_ptr is not copiable. + std::shared_ptr. It is not actually possible to just replace + PointerHolder with std::shared_ptr for two reasons: there is no + automatic creation of std::shared_ptr<T> from T* like there is for + PointerHolder, which breaks some code, and also there is no + automatic conversion from something like + std::vector<PointerHolder<T>> to std::vector<std::shared_ptr<T>>. It + may be a good idea to replace PointerHolder with std::shared_ptr in + the API even if it requires some work for the developer, but even if + that isn't worth it, we should find all occurrences of PointerHolder + within the code and replace with std::shared_ptr or std::unique_ptr + as needed. This will definitely break binary compatibility as the + PointerHolder<Members> pattern is part of the ABI for almost every + class. * QIntC.hh could be simplified with type_traits. |