- Dec 01, 2023
-
-
Sébastien Villemot authored
Automatically detected using clang-tidy with bugprone-reserved-identifier check. By the way, homogeneize the define identifiers in relation to camel case convention.
-
Sébastien Villemot authored
Automatically detected using clang-tidy with modernize-use-nodiscard check.
-
- Nov 30, 2023
-
-
Sébastien Villemot authored
-
- Jul 07, 2023
-
-
Sébastien Villemot authored
Initialization with initalizer list is nicer. And, more importantly, initialization with parentheses is not supported by Clang < 16.
-
- Sep 27, 2022
-
-
Sébastien Villemot authored
This is unsafe since the find() method can return a past-the-end iterator, which should be tested for. Replace most instances by calls to the std::map::at() method (which throws if the key is unknown), and which is incidentally more readable.
-
- Jul 20, 2022
-
-
Sébastien Villemot authored
By the way, turn those classes into structs since all their members are public.
-
- Jul 08, 2022
-
-
Sébastien Villemot authored
-
- Jun 07, 2022
-
-
Sébastien Villemot authored
-
- May 04, 2022
-
-
Sébastien Villemot authored
-
- Jan 06, 2022
-
-
Sébastien Villemot authored
Otherwise clang emits this warning: using directive refers to implicitly-defined namespace 'std'
-
- Jun 09, 2021
-
-
Sébastien Villemot authored
-
- Dec 16, 2019
-
-
Sébastien Villemot authored
-
- Apr 16, 2019
-
-
Sébastien Villemot authored
-
- Mar 18, 2019
-
-
Sébastien Villemot authored
-
- Oct 11, 2018
-
-
Sébastien Villemot authored
-
- Oct 04, 2018
-
-
Sébastien Villemot authored
The syntax makes clearer the distinction between a function call and an initialization. Also, it prevents bugs due to implicit type conversions.
-
Sébastien Villemot authored
For a rationale, see http://www.sjbrown.co.uk/2004/05/01/always-use-explicit/
-
- Oct 02, 2018
-
-
Sébastien Villemot authored
This was not conceptually an enum, but rather a collection of unrelated constants: - two constants for use as placeholder for symbol IDs at different places - one constant for the default number of arguments
-
- Jun 04, 2018
-
-
Sébastien Villemot authored
Those were not automatically replaced by clang-tidy, I don’t know why…
-
Sébastien Villemot authored
Performed using modernize-use-noexcept from clang-tidy. https://clang.llvm.org/extra/clang-tidy/checks/modernize-use-noexcept.html
-
Sébastien Villemot authored
Performed using modernize-use-auto from clang-tidy. https://clang.llvm.org/extra/clang-tidy/checks/modernize-use-auto.html
-
Sébastien Villemot authored
Performed using modernize-loop-convert from clang-tidy. Manual intervention was needed in MacroValue.cc because of a variable name capture issue. https://clang.llvm.org/extra/clang-tidy/checks/modernize-loop-convert.html
-
- Feb 15, 2018
-
-
Houtan Bastani authored
-
- May 16, 2017
-
-
Stéphane Adjemian authored
-
- Jun 09, 2015
-
-
Houtan Bastani authored
This reverts commit 28ffa9c56fd021cc75415aeafc13d3810d1b3132, reversing changes made to 60b3577a09f3eb9cf3cc590241343b1f0fee8706. Conflicts: preprocessor/Shocks.cc preprocessor/SymbolTable.cc
-
Houtan Bastani authored
This reverts commit def2bf81d2b8dacda4b72d4250020eb793f28b20.
-
- May 28, 2015
-
-
Houtan Bastani authored
-
- Apr 08, 2015
-
-
Houtan Bastani authored
-
- Feb 04, 2011
-
-
Sébastien Villemot authored
-
Sébastien Villemot authored
-
- Mar 03, 2010
-
-
Houtan Bastani authored
-
- Feb 23, 2010
-
-
Sébastien Villemot authored
-
Houtan Bastani authored
-