how-to
/
CppCoreGuidelines
mirror of https://github.com/isocpp/CppCoreGuidelines.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity
1,050 Commits
14 Branches
4 Tags
184 MiB
master
gh-pages
2170-hs
concrete-terminology
cast-to-void-cleanup
string_span-excision
For-1446
es23-updates
neilmacintosh-ES70-fix
hs-index
travis-errors
t84-example
revert-825-interfaces_non_trivial_fixes
H1
v0.7
v0.6
v0.1
v0.8
Branches Tags
${ item.name }
Create tag ${ searchTerm }
Create branch ${ searchTerm }
from '3768e82fc3'
${ noResults }
Commit Graph

1050 Commits (3768e82fc3b987e89a0cfacbbd77cbcaff4fdf30)
 

Author SHA1 Message Date
Titus Winters 3768e82fc3 Fix spelling / expand dictionary. 9 years ago
Titus Winters e8ecae3171 Added some example tooling discussion. 9 years ago
Gabriel Dos Reis 7476fc3194 Merge pull request #786 from mserdarsanli/master 
Fix broken anchor link
 9 years ago
Mustafa Serdar Sanli 53bc78fd49 Fix broken anchor link 9 years ago
Jan Nils Ferner d86a82f1eb Fix typo (#782) 9 years ago
Jonathan Wakely 98f1e46602 C.146 Compare pb2->id() instead of pb2 
Fixes #504
 9 years ago
Gabriel Dos Reis 173b036e51 Merge pull request #766 from congdanhqx/master 
ES.7 Fix typo, "i" -> "current_element_index"
 9 years ago
Doan Tran Cong Danh db85ec865d ES.7 Fix typo, "i" -> "current_element_index" 9 years ago
Jonathan Wakely aab97d9e48 C.32 Fix typo, "aide" -> "aid" 9 years ago
Gabriel Dos Reis 6767413118 Merge pull request #760 from derekxgl/patch-1 
P.11 call vector reserve(100) instead of vector(100)
 9 years ago
Gabriel Dos Reis 6819dd6f5f Merge pull request #761 from pepsiman/patch-2 
Correct spelling of nullptr
 9 years ago
Malcolm Parsons 38772fc251 Correct spelling of nullptr 9 years ago
Derek Li 6c29e226d8 P.11 call vector reserve(100) instead of vector(100) 9 years ago
Gabriel Dos Reis 7b4a1903ec Merge pull request #745 from kks32/master 
Fixes wrong order of type and function name in naming scheme - master branch
 9 years ago
Gabriel Dos Reis 804964d653 Merge pull request #758 from Eliyahu-Ravuna/patch-22 
Short string optimization example for C.180
 9 years ago
Eliyahu Ravuna 52c4376433 Short string optimization example for C.180 
Extra blank lines removed

strcpy replaced with strcpy_s to get rid of Travis CI warnings

Note: strcpy_s  is defined in the Annex K of the C++11 standard.

Gabriel's comments incorporated

- No need for braces for a single statement.
- Comment to explain why buffer_size is 16.

Bracing style made consistent with the rest of the examples

Bracing style made consistent with the rest of the examples

braces made consistent with the rest of the examples
 9 years ago
Andrew Pardoe f6dd6159f8 Fix issue #747 9 years ago
Andrew Pardoe f66d9aea46 Fix 798177568e 9 years ago
Andrew Pardoe 798177568e Merge pull request #754 from Eliyahu-Ravuna/patch-21 
CP.110 & CP.111: Double-checked locking
 9 years ago
Andrew Pardoe b149a50faf Merge pull request #756 from pepsiman/patch-1 
P.5 Correct case of Int alias
 9 years ago
Andrew Pardoe 4d45c6bb1d Merge pull request #755 from pepsiman/executionpolicy 
P.3 Split execution policy and container
 9 years ago
Eliyahu Ravuna ec2f23781d CP.110 renamed as "Do not write your own double-checked locking" 
CP.110 renamed as "Do not write your own double-checked locking".
Explanation and two examples added.

Added exception

Double-checked locking may be legitimately used in lock-free programming. However, see the rule [CP.100: Don't use lock-free programming unless you absolutely have to](#Rconc-lockfree).

Title made clearer and a conventional pattern has been added

* Title made clearer
* A correct example of double-checked locking has been added

Casing fixed

Casing fixed

Extra whitespace deleted

CP.110 and CP.111 separated

- CP.110: Do not write your own double-checked locking for initialization
- CP.111: Use a conventional pattern if you really need double-checked locking

Whitespace removed

Example simplified

Whitespace issue fixed.

punctuation fixed.
 9 years ago
Gabriel Dos Reis d23f9872e0 Merge pull request #751 from johelegp/introduction_fixes 
In: Grammar and correction fixes
 9 years ago
Malcolm Parsons 71df1537b3 P.5 Correct case of Int alias 
Revert d9562f683d and part of 9590bb94b1
 9 years ago
Malcolm Parsons e66f5bb28e P.3 Split execution policy and container 9 years ago
Johel Ernesto Guerrero Peña 2a000ac21a In: Grammar and correction fixes 
We use letters to identify the first part of a rule "number", not just one letter.
The rest are grammar fixes.
 9 years ago
Gabriel Dos Reis 21eb995573 Merge pull request #746 from blackjack/master 
Remove mode string from ifstream constructor in R.12
 9 years ago
Oleksandr Senkovych e0abec4259 Remove mode string from ifstream constructor in R.12 9 years ago
Krishna Kumar 7f872a9d50 Fixes wrong order of type and function name in naming scheme 9 years ago
Gabriel Dos Reis 2183f77ac7 Merge pull request #743 from johelegp/glossary_fixes 
Correct glossary entry order
 9 years ago
Johel Ernesto Guerrero Peña 8aaf6e1b76 Correct glossary entry order 9 years ago
Gabriel Dos Reis e57db552e4 Merge pull request #740 from johelegp/glossary_fixes 
Fix glossary definitions and grammar
 9 years ago
Johel Ernesto Guerrero Peña 138da22db4 Fix glossary definitions and grammar 
A class isn't abstract for having a protected constructor, but for only having protected constructors.
 9 years ago
Gabriel Dos Reis 1894380d0a Merge pull request #737 from rianquinn/patch-1 
Update C.128 to remove override from destructors
 9 years ago
Andrew Pardoe a75e33abc5 Merge pull request #736 from tkruse/git-conflict 
Remove merged git conflict
 9 years ago
Rian Quinn 8be1869663 Update C.128 to remove override from destructors 
Clang Tidy has a a check called (modernize-use-override) that explicitly verifies that `override` be placed on destructors of derived classes whose base class is `virtual` as seen [here](https://github.com/Microsoft/clang-tools-extra/blob/master/test/clang-tidy/modernize-use-override.cpp#L48). This issue was brought up by @jaredgrubb in the following [ticket](https://github.com/isocpp/CppCoreGuidelines/issues/721#issuecomment-246627077) and was also seen [here](https://github.com/Bareflank/hypervisor/issues/208) as well. @gdr-at-ms closed the ticket stating that the C++ Core Guideline Editors have decided that `override` should not be placed on destructors, but the documentation makes no mention of this decision. The following PR addresses this issue. With the documentation updated, an issue ticket can be generated for Clang Tidy to have the destructor check modified to reflect the C++ Core Guidance.
 9 years ago
Thibault Kruse 13efb300b1 Remove merged git conflict 9 years ago
Andrew Pardoe 157e7a9d9e Merge branch 'tkruse-python-fix' 9 years ago
Andrew Pardoe 61717ae4eb Merge branch 'python-fix' of https://github.com/tkruse/CppCoreGuidelines into tkruse-python-fix 9 years ago
Gabriel Dos Reis 3f33bd9c89 Merge pull request #730 from olafdietsche/spelling 
Minor spelling fixes
 9 years ago
Gabriel Dos Reis 729c515a2b Merge pull request #703 from amdn/master 
T.44: Added `s` suffix to string literal
 9 years ago
Andrew Pardoe 128a3e1401 Merge pull request #707 from tkruse/stricter-cpplint-checks 
stricter cpplint checks
 9 years ago
Andrew Pardoe c4a19c99b2 Merge pull request #706 from tkruse/fix-examples2 
Fix examples2
 9 years ago
Andrew Pardoe 66c15ba47c Merge pull request #705 from tkruse/fix-spellchecks 
Fix markdown checks and spell checks
 9 years ago
Andrew Pardoe 97503abeaa Merge pull request #700 from tkruse/fix-examples 
fix code examples
 9 years ago
Gabriel Dos Reis 2f1fd85b9e Merge pull request #731 from akimd/fix-t.65 
T.65: fix code examples
 9 years ago
Akim Demaille 2b80bc92a5 T.65: fix code examples 9 years ago
Olaf Dietsche bb2e744c3f Minor spelling fixes 9 years ago
Thibault Kruse 5a02b22bb2 stricter cpplint checks 9 years ago
Thibault Kruse 1c0e2b7d11 fix invalid code 9 years ago