search

sftrabbit / CppPatterns-Patterns

A repository of modern C++ patterns curated by the community.
Creative Commons Zero v1.0 Universal
1.43k stars 223 forks source link

Adds missing link. #41

Closed dyp-cpp closed 9 years ago

dyp-cpp commented 9 years ago

Link missing from second mention of shared_ptr.

sftrabbit commented 9 years ago

I've actually been doing this on purpose - I tend to like linking the first instance of a term but not the rest. Not sure what others think of this.

dyp-cpp commented 9 years ago

Ah, well I thought there are the brackets around std::shared_ptr, and you have linked weak_ptr twice, so it seemed a bit inconsistent. I concur with linking only the first occurrence, but what's the purpose of the [] then?

sftrabbit commented 9 years ago

Oh, I didn't notice that, thanks!