StarMirror/CppCoreGuidelines
1
0
Fork 0
mirror of https://github.com/isocpp/CppCoreGuidelines.git synced 2024-03-22 13:30:58 +08:00
Code Issues Projects Releases Wiki Activity

T.61 example fixes (#1813)

Browse Source
This commit is contained in:
ash 2021-07-29 23:52:06 +05:00 committed by GitHub
parent fa726d5b0a
commit 5bbf843942
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
CppCoreGuidelines.md
View File

@ -18034,14 +18034,14 @@ This limits use and typically increases code size.
##### Example, bad
template<typename T, typename A = std::allocator{}>
template<typename T, typename A = std::allocator<T>>
// requires Regular<T> && Allocator<A>
class List {
public:
struct Link { // does not depend on A
T elem;
T* pre;
T* suc;
Link* pre;
Link* suc;
};
using iterator = Link*;
@ -18062,11 +18062,11 @@ Typically, the solution is to make what would have been a nested class non-local
template<typename T>
struct Link {
T elem;
T* pre;
T* suc;
Link* pre;
Link* suc;
};
template<typename T, typename A = std::allocator{}>
template<typename T, typename A = std::allocator<T>>
// requires Regular<T> && Allocator<A>
class List2 {
public:
@ -18076,11 +18076,11 @@ Typically, the solution is to make what would have been a nested class non-local
// ...
private:
Link* head;
Link<T>* head;
};
List<int> lst1;
List<int, My_allocator> lst2;
List2<int> lst1;
List2<int, My_allocator> lst2;
Some people found the idea that the `Link` no longer was hidden inside the list scary, so we named the technique
[SCARY](http://www.open-std.org/jtc1/sc22/WG21/docs/papers/2009/n2911.pdf). From that academic paper: