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,020 Commits 14 Branches 4 Tags 216 MiB
8aaf6e1b76
Commit Graph

1020 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Johel Ernesto Guerrero Peña
8aaf6e1b76 Correct glossary entry order 2016-09-27 13:42:22 -04:00
Gabriel Dos Reis
e57db552e4 Merge pull request #740 from johelegp/glossary_fixes 
Fix glossary definitions and grammar
 2016-09-27 08:00:11 -07: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
Gabriel Dos Reis
1894380d0a Merge pull request #737 from rianquinn/patch-1 
Update C.128 to remove override from destructors
 2016-09-26 11:39:30 -07:00
Andrew Pardoe
a75e33abc5 Merge pull request #736 from tkruse/git-conflict 
Remove merged git conflict
 2016-09-26 10:58:06 -07: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
Thibault Kruse
13efb300b1 Remove merged git conflict 2016-09-13 06:42:50 +09:00
Andrew Pardoe
157e7a9d9e Merge branch 'tkruse-python-fix' 2016-09-12 11:54:09 -07:00
Andrew Pardoe
61717ae4eb Merge branch 'python-fix' of https://github.com/tkruse/CppCoreGuidelines into tkruse-python-fix 2016-09-12 11:53:55 -07: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
128a3e1401 Merge pull request #707 from tkruse/stricter-cpplint-checks 
stricter cpplint checks
 2016-09-12 11:11:41 -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
66c15ba47c Merge pull request #705 from tkruse/fix-spellchecks 
Fix markdown checks and spell checks
 2016-09-12 11:08:47 -07:00
Andrew Pardoe
97503abeaa Merge pull request #700 from tkruse/fix-examples 
fix code examples
 2016-09-12 11:08:13 -07:00
Gabriel Dos Reis
2f1fd85b9e Merge pull request #731 from akimd/fix-t.65 
T.65: fix code examples
 2016-09-12 09:55:28 -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
5a02b22bb2 stricter cpplint checks 2016-09-10 22:56:25 +09: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
Thibault Kruse
d918525609 python: fix superfluous newline in extracted plain.txt, causes bad line numbers in spell check 2016-09-10 22:56:05 +09:00
Thibault Kruse
690f9e9aed repair markdown checking 2016-09-10 22:56:05 +09:00
Thibault Kruse
084eb29d42 minor improvement, strip newlines from end of code snippets 2016-09-10 22:55:58 +09:00
Thibault Kruse
1842b88c21 refactor python code extractor: extract function 2016-09-10 22:55:58 +09:00
Thibault Kruse
8e282fa90c Minor fix to python code extractor, dedent by indentation amount 2016-09-10 22:55:58 +09:00
Thibault Kruse
154dd24a54 improve cpplint check by invoking python just once. Reduces time from 1m to 6s 2016-09-10 22:55:58 +09:00
Gabriel Dos Reis
3acba2ebae Merge pull request #726 from tkruse/fix-typos 
Fix typos
 2016-09-09 19:23:25 -07:00
Gabriel Dos Reis
a051b7e840 Merge pull request #728 from olafdietsche/issue_cp_3 
CP.3: Fix example: function declarations
 2016-09-09 11:44:42 -07:00
Olaf Dietsche
bceec9e494 CP.3: Fix example: function declarations 2016-09-09 18:44:19 +02:00
Thibault Kruse
da31c7dacf extend dict 2016-09-07 18:18:10 +09:00
Thibault Kruse
a11f1f7f53 fix typos 2016-09-07 18:17:59 +09:00
Gabriel Dos Reis
031379332d Merge pull request #724 from fmorgner/master 
Move whitespace to right place
 2016-09-06 14:58:21 -07:00
Felix Morgner
8078b0a1f8 Move whitespace to right place 2016-09-06 15:43:55 +02:00
Gabriel Dos Reis
fdc4a6df04 Merge pull request #722 from tkruse/fix-text 
Fix text mistake
 2016-09-06 02:47:26 -07: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
Gabriel Dos Reis
db029855dd Merge pull request #696 from tkruse/style-fixes 
Style fixes
 2016-09-06 01:14:20 -07: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
Gabriel Dos Reis
443c1a4afb Merge pull request #719 from VeXocide/patch-1 
Fixes the NL.26 link.
 2016-08-31 16:46:35 -07:00
Jeroen Habraken
bf0d7e415f Fixes the NL.26 link. 2016-08-31 16:32:34 -07:00