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
1,084 Commits 14 Branches 4 Tags 216 MiB
5f95cb2734
Commit Graph

848 Commits

Author SHA1 Message Date
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.
 2016-10-01 19:18:50 -04:00
Oleksandr Senkovych
e0abec4259 Remove mode string from ifstream constructor in R.12 2016-09-28 11:23:18 +02:00
Krishna Kumar
7f872a9d50 Fixes wrong order of type and function name in naming scheme 2016-09-27 15:43:04 -07:00
Johel Ernesto Guerrero Peña
8aaf6e1b76 Correct glossary entry order 2016-09-27 13:42:22 -04:00
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.
 2016-09-27 02:00:24 -04:00
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.
 2016-09-13 07:05:52 -06:00
Gabriel Dos Reis
3f33bd9c89 Merge pull request #730 from olafdietsche/spelling 
Minor spelling fixes
 2016-09-12 11:48:55 -07:00
Gabriel Dos Reis
729c515a2b Merge pull request #703 from amdn/master 
T.44: Added `s` suffix to string literal
 2016-09-12 11:13:04 -07:00
Andrew Pardoe
c4a19c99b2 Merge pull request #706 from tkruse/fix-examples2 
Fix examples2
 2016-09-12 11:11:16 -07:00
Andrew Pardoe
97503abeaa Merge pull request #700 from tkruse/fix-examples 
fix code examples
 2016-09-12 11:08:13 -07:00
Akim Demaille
2b80bc92a5 T.65: fix code examples 2016-09-12 18:41:54 +02:00
Olaf Dietsche
bb2e744c3f Minor spelling fixes 2016-09-11 17:19:52 +02:00
Thibault Kruse
1c0e2b7d11 fix invalid code 2016-09-10 22:56:15 +09:00
Thibault Kruse
25e3ec4652 improve code example, use() function makes little sense with void results of f() and g() 2016-09-10 22:56:15 +09:00
Thibault Kruse
7c991f0e7e fix varname 2016-09-10 22:56:15 +09:00
Thibault Kruse
a361c37f5e unique funIds 2016-09-10 22:56:15 +09:00
Thibault Kruse
9590bb94b1 fix code examples 2016-09-10 22:56:14 +09:00
Thibault Kruse
2db47928df fix unnecessary block 2016-09-10 22:56:11 +09:00
Thibault Kruse
6a39a88bf8 Fix initializer list example 2016-09-10 22:56:11 +09:00
Gabriel Dos Reis
3acba2ebae Merge pull request #726 from tkruse/fix-typos 
Fix typos
 2016-09-09 19:23:25 -07:00
Olaf Dietsche
bceec9e494 CP.3: Fix example: function declarations 2016-09-09 18:44:19 +02:00
Thibault Kruse
a11f1f7f53 fix typos 2016-09-07 18:17:59 +09:00
Felix Morgner
8078b0a1f8 Move whitespace to right place 2016-09-06 15:43:55 +02:00
Thibault Kruse
1e4b13bfb0 fix whitespace after comma 2016-09-06 18:32:23 +09:00
Thibault Kruse
9c148afad3 Fix transition of *Excption* to #### Exception in 4d7b1d 2016-09-06 18:31:47 +09:00
Thibault Kruse
254c123c14 fix internal link 2016-09-05 22:22:08 +09:00
Thibault Kruse
13419aa5dd fix code style 2016-09-05 22:17:03 +09:00
Thibault Kruse
603a1b4286 Fix whitespace inconsistencies, remove tabs 2016-09-05 22:08:04 +09:00
Thibault Kruse
27b585ad71 style issues 2016-09-05 22:08:04 +09:00
Bjarne Stroustrup
0aea7d997b Added Per.7 
Design to enable optimization
 2016-09-04 11:08:54 -04:00
Jeroen Habraken
bf0d7e415f Fixes the NL.26 link. 2016-08-31 16:32:34 -07:00
Neil MacIntosh
b5d5d3f547 Fix order of declarations inside example 
Fixes #668.
 2016-08-30 14:31:16 -07:00
Neil MacIntosh
d1f22ae88d Fix order of declarations inside example 
As per #668, fixing order of declarations in example to match NL.16.
 2016-08-30 14:22:40 -07:00
Bjarne Stroustrup
3ca23f1575 fix Enum.4 2016-08-29 15:50:47 -04:00
Bjarne Stroustrup
3ec8cfc5cc Issue 660 
Note added
 2016-08-28 14:49:47 -04:00
Bjarne Stroustrup
4d7b1d7b63 minor formatting changes 2016-08-28 14:39:49 -04:00
Bjarne Stroustrup
56b4efd430 fixing some bugs in examples 
#697
 2016-08-28 12:31:11 -04:00
Thibault Kruse
4d97aea4c9 fix typos 2016-08-28 11:58:16 +09:00
Bjarne Stroustrup
d7725aa37c ES.100 
new example
 2016-08-27 21:14:16 -04:00
Bjarne Stroustrup
1d53754694 Union section fleshed out 
#692 #570
 2016-08-27 14:49:57 -04:00
Bjarne Stroustrup
cbad9769de ES.102 fleshed out 
response to issue #675
 2016-08-26 16:30:20 -04:00
Bjarne Stroustrup
34e719bb36 ES.12: Do not reuse names in nested scopes 
new rule, and a few minor fixes
 2016-08-26 11:48:09 -04:00
Gabriel Dos Reis
ca798a640b Merge pull request #709 from tkruse/fix-typos 
Fix typos
 2016-08-25 20:02:23 -07:00
Thibault Kruse
83f60b4415 Fix typos 2016-08-26 04:59:00 +02:00
Thibault Kruse
5aea4a1fef style: fix code indentation 2016-08-26 04:03:20 +02:00
Bjarne Stroustrup
67191255fa Updates to NR section 2016-08-25 16:18:48 -04:00
Arturo Martín-de-Nicolás
d01675876f Added s suffix to string literal 
As written, the type for "Hamlet" without the `s` suffix will be deduced to `const char*` instead of the intended `std::string`, see example earlier for auto deduced `std::string` for "Ophelia"s with the `s` suffix.
 2016-08-24 16:01:48 -05:00
Bjarne Stroustrup
1ddd9cdb7a typo fix 
P.11 issue #670
 2016-08-24 11:18:58 -04:00
Andrew Pardoe
3f0aa7c379 Merge pull request #694 from tkruse/merge-leftovers 
Bad Merge leftovers
 2016-08-22 18:00:04 -07:00
Thibault Kruse
1931fdcff5 Google styleguide now lives on github 2016-08-23 00:15:42 +02:00
Thibault Kruse
a1c248b26f unique variable names in example 2016-08-22 23:21:14 +02:00
Thibault Kruse
c0bff45961 style fixes 2016-08-22 23:21:14 +02:00
Andrew Pardoe
9cf040ebec Merge branch 'fix-style-classname' of https://github.com/tkruse/CppCoreGuidelines into tkruse-fix-style-classname 2016-08-22 12:00:11 -07:00
Andrew Pardoe
fbe6aac021 Merge pull request #618 from tlanc007/CP.2_617 
CP.2_617: Changed variable s to a tigher form of span.
 2016-08-22 11:43:39 -07:00
Gabriel Dos Reis
6760960e7a Merge pull request #613 from tlanc007/C.33_610 
C.33_610: changed owner<T>* to owner<T*> per issue 610
 2016-08-22 11:39:31 -07:00
Gabriel Dos Reis
4efa0fb291 Merge pull request #666 from tkruse/dodgy-code 
Minor dodgy code issues
 2016-08-22 11:33:01 -07:00
Andrew Pardoe
7cbdb3d356 Merge branch 'fix-unique-anchors' of https://github.com/tkruse/CppCoreGuidelines into tkruse-fix-unique-anchors 2016-08-22 11:27:12 -07:00
Gabriel Dos Reis
7e34d8880c Merge pull request #681 from tkruse/rebased-typo-fixes 
Rebased typo fixes
 2016-08-22 11:25:56 -07:00
Andrew Pardoe
bfcffb8561 Merge branch 'master' of https://github.com/0x8000-0000/CppCoreGuidelines into 0x8000-0000-master 2016-08-22 11:16:42 -07:00
Andrew Pardoe
3f3d80d7ec Reapplying one fix from 680 after a slightly bad merge 2016-08-22 11:11:10 -07:00
Andrew Pardoe
6ba822f535 Merge branch 'upgrade-checkers' of https://github.com/tkruse/CppCoreGuidelines into tkruse-upgrade-checkers 2016-08-22 11:09:01 -07:00
Andrew Pardoe
7f510892e5 Merge pull request #680 from tkruse/fix-style 
style fixes
 2016-08-22 11:05:40 -07:00
Andrew Pardoe
9a9e958955 Manually fixed PR #657 2016-08-22 10:49:29 -07:00
Andreas Scherer
25044afca0 Correct a few typos. 
A quick inspection with 'aspell -l en_US'.
 2016-08-21 12:26:42 +02:00
Thibault Kruse
be304fedd6 R.1 example: change names according to NL.8 2016-08-18 02:00:36 +02:00
Thibault Kruse
0af83def79 Style: Rename classes with underscore as separator 2016-08-18 01:49:26 +02:00
Thibault Kruse
2bfb860cfe separate names with underscores 2016-08-18 01:47:12 +02:00
Thibault Kruse
0701c4df62 Style: Consistently name custom classes with initial capital and underscore notation 2016-08-18 01:33:29 +02:00
Thibault Kruse
00cf8d08ff Fix bad reference 2016-08-17 22:25:49 +02:00
Sam Bristow
9f2d2645ff Remove rogue comma from example 2016-08-17 18:29:08 +02:00
Passw
04cd9a5d73 fix typo 2016-08-17 18:28:24 +02:00
Robin Degen
d084dcef31 Fixed typo shuld -> should, fixed missing words 2016-08-17 18:26:59 +02:00
Thibault Kruse
70d056f087 Fix duplicate/wrong anchors. Fix Makefile uniq check for anchors, must sort 2016-08-17 18:05:33 +02:00
Thibault Kruse
4cbbf55bd3 style fixes 2016-08-17 18:01:58 +02:00
Thibault Kruse
6e1599f6f9 style fixes 2016-08-17 17:55:51 +02:00
Thibault Kruse
0120560f13 update to pypi cpplint 1.3.0, fix linter errors 2016-08-17 17:53:57 +02:00
Thibault Kruse
ced78ffad1 typo 2016-08-17 17:53:44 +02:00
Thibault Kruse
6867d13363 type error 2016-08-17 17:53:43 +02:00
Thibault Kruse
9160dbb818 invalid enums 2016-08-17 17:53:43 +02:00
Thibault Kruse
b14fe453a4 avoid keyword name 2016-08-17 17:53:43 +02:00
Thibault Kruse
9d4fc0b5cb bad type 2016-08-17 17:53:43 +02:00
Thibault Kruse
d6ffbfdcc2 missing semicolon 2016-08-17 17:53:43 +02:00
Thibault Kruse
3e1519beb3 bad parens within name 2016-08-17 17:53:43 +02:00
Thibault Kruse
bcca1488e8 unique var names in example 2016-08-17 17:53:43 +02:00
Thibault Kruse
46a26976ba typo in method invocation 2016-08-17 17:53:43 +02:00
Thibault Kruse
4e46bd9a8b fix indent 2016-08-17 17:53:43 +02:00
Thibault Kruse
12bdb63b06 fix parens 2016-08-17 17:53:42 +02:00
Thibault Kruse
6a8728a054 qualified access 2016-08-17 17:53:42 +02:00
Thibault Kruse
04345df6d0 remove duplicate constructor 2016-08-17 17:53:42 +02:00
Thibault Kruse
0997489573 bad signature 2016-08-17 17:53:42 +02:00
Thibault Kruse
b9f41b5cd0 missing template argument 2016-08-17 17:53:42 +02:00
Thibault Kruse
0b15a43c54 missing comma 2016-08-17 17:53:42 +02:00
Thibault Kruse
dc683521ee wrong sample class 2016-08-17 17:53:42 +02:00
Thibault Kruse
01b66d9a7e unique variable names in example 2016-08-17 17:53:42 +02:00
Thibault Kruse
bf11606c1a completely implement assignOperator 2016-08-17 17:53:42 +02:00
Thibault Kruse
165c61e7ce bad signature 
# Conflicts:
#	CppCoreGuidelines.md
 2016-08-17 17:53:42 +02:00
Thibault Kruse
005e546d63 bad return type 2016-08-17 17:53:41 +02:00
Thibault Kruse
2a9d0a43b8 dodgy example code 2016-08-17 17:53:41 +02:00
Thibault Kruse
360ba610d1 Fix many typos 2016-08-17 17:40:13 +02:00
Gabriel Dos Reis
d4da142302 Merge pull request #678 from alex-weej/patch-3 
Typos in NL.7
 2016-08-17 08:35:06 -07:00