Errata

Error:
class Person3
{
//...

private:
// ...
struct Impl;
std::unique_ptr<Pimpl> pimpl_; // 'Pimpl' template param should be 'Impl'
};

Correct
class Person3
{
//...

private:
// ...
struct Impl;
std::unique_ptr<Impl> pimpl_;
};

Note from the Author or Editor:
Thanks a lot for taking the time to submit this error. Yes, the 'Pimpl' in the std::unique_ptr should be an 'Impl'. This will be fixed in the next release of the book.

Edit: The fix has been committed to the repository.

The constructor of CppBook takes 3 arguments but is only given 2.

Note from the Author or Editor:
The finding is correct: in the code example on page 21 the CppBook instances should take three arguments instead of just 2. The fix is either to change the constructor of the CppBook class to only take two parameters or to fix and reformat the code example to pass 3 parameters.

Error:
template< std::forward_iterator ForwardIt, std::forward_iterator ForwardIt >
OutputIt copy( ForwardIt first, ForwardIt last, ForwardIt d_first );

Possible solution:
template< std::forward_iterator ForwardIt, std::forward_iterator OutputIt >
OutputIt copy(ForwardIt first, ForwardIt last, OutputIt d_first);

Note from the Author or Editor:
Thanks for taking the time to report this error. Yes, you are correct, the copy function should take two different kinds of iterators. The example should be like this:

template< std::forward_iterator ForwardIt >
OutputIt copy(ForwardIt first, ForwardIt last, ForwardIt d_first);

I apologize for this oversight. This error will be fixed in the next print of the book. Thanks again,

Best regards,
Klaus!

Edit: The fix has been committed to the repository.

auto strategy = std::make_unique_ptr<...
is an error/typo. It should be
auto strategy = std::make_unique<...

Note from the Author or Editor:
This is indeed a typo in a code example. The fix has already been merged into the repo.

Instead of DrawCircleStrategy should it not be DrawStrategy<Circle>
The code for the figure is correct.

Note from the Author or Editor:
Figure 5-6 should indeed contain DrawStrategy<Circle> instead of DrawCircleStrategy. Therefore this is a minor inconsistence between the image and the code example(s). This will have to be fixed before the next release of the book.

Instead of undo() function it should be undoLast() function.

Note from the Author or Editor:
This is a small inconsistency between the code examples and the accompanying text. The current sentence is:

"As a result, both compute() and undo() do not have to be virtual functions."

It should be

"As a result, both compute() and undoLast() do not have to be virtual functions."

Based on the previous codeblock, the import part should not been comment out.

So,
```
//#include <memory>
```
Should be
```
#include <memory>
```

This can also apply to the next page's "Person.cpp" block, where
```
//#include <Person.h>
//#include <string>
```
Should be
```
#include <Person.h>
#include <string>
```

Note from the Author or Editor:
The finding is correct: All three mentioned includes are incorrectly commented out. These comments should be removed.

Isn't Table 7-2 heading should be for compilers GCC11.1 and Clang 11.1 as in Table 7-1 ?

Note from the Author or Editor:
It is correct that the headings for Table 7-1 and 7-2 are different. Once the book is in reprint, these headings will have to be checked again and will have to be adapted.

In the figure 7-5, it has an arrow at the top which associate with text "Engine". In the original picture of GoF book (page 119), this text should be "prototype".

Note from the Author or Editor:
This finding is correct: the arrow should be labeled with "prototype" instead of "Engine". This seems to be a copy and paste error from other images.

pimpl_(clone_(…), …) should be pimpl_(other.clone_(…), …) since clone_ is uninitialized when we initialize the unique_ptr pimpl_. The official github file G33_Small_Buffer_Optimization.cpp also supports this fix.

Note from the Author or Editor:
The finding is correct, the code example needs to be updated. The fix has already been merged into the repo on March 8th.

Error:

template< typename DocumentT >
class DocumentModel
{
public:
// ...


Correct:

template< typename DocumentT >
class DocumentModel : public DocumentConcept
{
public:
// ...

Note from the Author or Editor:
Thanks a lot for taking the time to submit this error. You are correct, the 'DocumentModel' should inherit from the 'DocumentConcept'. This will be fixed in the next release of the book.

Edit: The fix has been committed to the repository.