| Document number |
P2657R0 |
| Date |
2022-10-11 |
| Reply-to |
Jarrad J. Waterloo <descender76 at gmail dot com>
|
| Audience |
Evolution Working Group (EWG) |
C++ is the next C++
Table of contents
Abstract
Programmer’s, Businesses and Government(s) want C++ to be safer and simpler. This has led some C++ programmers to create new programming languages or preprocessors, which again is a new language. This paper discusses using static analysis to make the C++ language itself safer and simpler.
Motivating Examples
Following is a wishlist. Most are optional. While, they all would be of benefit. It all starts with a new repeatable module level attribute that would preferably be applied once in the primary module interface unit and would automatically apply to it and all module implementation unit(s). It could also be applied to a module implementation unit but that would generally be less useful. However, it might aid in gradual migration.
export module some_module_name [[static_analysis("")]];
module some_module_name [[static_analysis("")]];
- It would be ideal if the name member of the
static_analysis attribute could be passed as either an environment variable and/or command line argument to compilers so it could be used by pipelines to assert the degree of conformance to the defined static analyzer without actually changing the source.
- It would be ideal if compilers could standardize the environment variable name or command line argument name in order to ease tooling.
- It would be ideal if compilers could produce a machine readable report in JSON, YAML or something else so that pipelines could more easily consume the results.
- It would be ideal if compilers could standardize the machine readable report.
The name of the static analyzer are dotted. Unscoped or names that start with std., c++., cpp., cxx. or c. are reserved for standardization.
This proposal wants to stardardize two overarching static analyzer names; safer and modern.
[[static_analysis("safer")]]
|
The safer analyzer is for safety, primarily memory related. It is for those businesses and programmers who must conform to safety standards.
|
[[static_analysis("modern")]]
|
The safer analyzer is a subset of modern analyzer. The modern analyzer goes beyond just memory and safety concerns. It can be thought of as bleeding edge. It is for those businesses and programmers who commit to safety and higher quality modern code.
|
Neither is concerned about formatting or nitpicking. Both static analyzers only produce errors. These are meant for programmers, businesses and governments in which safety takes precedence. They both represent +∞. When a new version of the standard is released and adds new sub static analyzers than everyone’s code is broken, until their code is fixed. These sub static analyzers usually consist of features that have been mostly replaced with some other feature. It would be ideal if the errors produced not only say that the code is wrong but also provide a link to html page(s) maintained by the C++ teaching group, the authors of the C++ Core Guidelines and compiler specific errors. These pages should provide example(s) of what is being replaced and by what was it replaced. Mentioning the version of the C++ standard would also be helpful.
All modules can be used even if they don’t use the static_analysis attribute as this allows gradual adoption.
What are the safer and modern analyzers composed of?
These overarching static analyzers are composed of multiple static analyzers which can be used individually to allow a degree of gradual adoption.
Use lvalue references
[[static_analysis("use_lvalue_references")]]
|
use_lvalue_references is a subset of safer.
|
- Any declaration of a pointer is an error.
- Calling any function that has parameters that take a pointer is an error unless the pointer type are “pointer to
const character type” or “const pointer to const character type” and their arguments were string literals.
- string literals are always safe having static storage duration
std::string and std::string_view must be creatable at compile time
- Function pointers and member function pointers can still be used.
- Function pointers and member function pointers that declare pointers to non [member] function pointers produces an error.
- Lvalue references, &, can still be used.
WHY?
- A large portion of the C++ community have been programming without pointers for years. Some can go their whole career this way. This proposal just standardise existing practice.
- Modern
C++ has been advocated to programmers in other programming languages who complain about memory issues. This allows us to show them what we have been saying for decades.
- Over half of our memory related issues gets hashed away.
- Pointers have largely been replaced with the following:
|
lvalue references
|
1985: Cfront 1.0
|
|
STL
|
1992
|
|
std::unique_ptr, std::shared_ptr, std::weak_ptr, std::reference_wrapper, std::make_shared
|
C++11
|
|
std::make_unique
|
C++14
|
|
std::string_view, std::optional, std::any, std::variant
|
C++17
|
|
std::make_shared support arrays, std::span
|
C++20
|
The C++ Core Guidelines identifies issues that this feature helps to mitigate.
P.4: Ideally, a program should be statically type safe
P.6: What cannot be checked at compile time should be checkable at run time
P.7: Catch run-time errors early
P.8: Don’t leak any resources
P.11: Encapsulate messy constructs, rather than spreading through the code
P.12: Use supporting tools as appropriate
P.13: Use support libraries as appropriate
I.4: Make interfaces precisely and strongly typed
I.11: Never transfer ownership by a raw pointer (T*) or reference (T&)
I.12: Declare a pointer that must not be null as not_null
I.13: Do not pass an array as a single pointer
I.23: Keep the number of function arguments low
F.7: For general use, take T* or T& arguments rather than smart pointers
F.15: Prefer simple and conventional ways of passing information
F.22: Use T* or owner<T*> to designate a single object
F.23: Use a not_null<T> to indicate that “null” is not a valid value
F.25: Use a zstring or a not_null<zstring> to designate a C-style string
F.26: Use a unique_ptr<T> to transfer ownership where a pointer is needed
F.27: Use a shared_ptr<T> to share ownership
F.42: Return a T* to indicate a position (only)
F.43: Never (directly or indirectly) return a pointer or a reference to a local object
C.31: All resources acquired by a class must be released by the class’s destructor
C.32: If a class has a raw pointer (T*) or reference (T&), consider whether it might be owning
C.33: If a class has an owning pointer member, define a destructor
C.149: Use unique_ptr or shared_ptr to avoid forgetting to delete objects created using new
C.150: Use make_unique() to construct objects owned by unique_ptrs
C.151: Use make_shared() to construct objects owned by shared_ptrs
R.1: Manage resources automatically using resource handles and RAII (Resource Acquisition Is Initialization)
R.2: In interfaces, use raw pointers to denote individual objects (only)
R.3: A raw pointer (a T*) is non-owning
R.5: Prefer scoped objects, don’t heap-allocate unnecessarily
R.10: Avoid malloc() and free()
R.11: Avoid calling new and delete explicitly
R.12: Immediately give the result of an explicit resource allocation to a manager object
R.13: Perform at most one explicit resource allocation in a single expression statement
R.14: Avoid [] parameters, prefer span
R.15: Always overload matched allocation/deallocation pairs
R.20: Use unique_ptr or shared_ptr to represent ownership
R.22: Use make_shared() to make shared_ptrs
R.23: Use make_unique() to make unique_ptrs
ES.20: Always initialize an object
ES.24: Use a unique_ptr<T> to hold pointers
ES.42: Keep use of pointers simple and straightforward
ES.47: Use nullptr rather than 0 or NULL
ES.60: Avoid new and delete outside resource management functions
ES.61: Delete arrays using delete[] and non-arrays using delete
ES.65: Don’t dereference an invalid pointer
E.13: Never throw while being the direct owner of an object
CPL.1: Prefer C++ to C
Gotchas
Usage of smart pointers
This static analyzer causes programmers to use 2 extra characters when using smart pointers, -> vs (*)., since the overloaded -> operator returns a pointer.
smart_pointer->some_function();
|
(*smart_pointer).some_function();
|
the main function and environment variables
A shim module is needed in order to transform main and env functions into a more C++ friendly functions. These have been asked for years.
A Modern C++ Signature for main
Desert Sessions: Improving hostile environment interactions
No unsafe casts
[[static_analysis("no_unsafe_casts")]]
|
no_unsafe_casts is a subset of safer.
|
- Using
C/core cast produces an error.
- Using
reinterpret_cast produces an error.
- Using
const_cast produces an error.
Why?
C/core cast was replaced by static_cast and dynamic_cast.
- The
reinterpret_cast is needed more for library authors than their users. For library users it usually just causes problems and questions. It is rarely used in daily C++ when coding at a higher level.
- The
const_cast is needed more for library authors than their users. It is a means for the programmer to lie to oneself. For library users it usually just causes problems and questions. It is rarely used in daily C++ when coding at a higher level.
The C++ Core Guidelines identifies issues that this feature helps to mitigate.
C.146: Use dynamic_cast where class hierarchy navigation is unavoidable
ES.48: Avoid casts
ES.49: If you must use a cast, use a named cast
ES.50: Don’t cast away const
No unions
[[static_analysis("no_union")]]
|
no_union is a subset of safer.
|
- Using the
union keyword produces an error.
It was replaced by std::variant, which is safer.
The C++ Core Guidelines identifies issues that this feature helps to mitigate.
C.181: Avoid “naked” unions
No mutable
[[static_analysis("no_mutable")]]
|
no_mutable is a subset of safer.
|
- Using the
mutable keyword produces an error.
The programmer shall not lie to oneself. The mutable keyword violates the safety of const and is rarely used at a high level.
No new or delete
[[static_analysis("no_new_delete")]]
|
no_new_delete is a subset of safer.
|
- Using the
new and delete keywords to allocate and deallocate memory produces an error.
It was replaced by std::make_unique and std::make_shared, which are safer.
The C++ Core Guidelines identifies issues that this feature helps to mitigate.
F.26: Use a unique_ptr<T> to transfer ownership where a pointer is needed
F.27: Use a shared_ptr<T> to share ownership
C.149: Use unique_ptr or shared_ptr to avoid forgetting to delete objects created using new
C.150: Use make_unique() to construct objects owned by unique_ptrs
C.151: Use make_shared() to construct objects owned by shared_ptrs
R.11: Avoid calling new and delete explicitly
R.20: Use unique_ptr or shared_ptr to represent ownership
R.22: Use make_shared() to make shared_ptrs
R.23: Use make_unique() to make unique_ptrs
ES.60: Avoid new and delete outside resource management functions
ES.61: Delete arrays using delete[] and non-arrays using delete
No volatile
[[static_analysis("no_volatile")]]
|
no_volatile is a subset of safer.
|
- Using the
volatile keyword produces an error.
The volatile keyword has nothing to do with concurrency. Use std::atomic or std::mutex instead.
The C++ Core Guidelines identifies issues that this feature helps to mitigate.
CP.8: Don’t try to use volatile for synchronization
No C style variadic functions
[[static_analysis("no_c_style_variadic_functions")]]
|
no_c_style_variadic_functions is a subset of safer.
|
- Declaring a
C style variadic function produces an error.
- Calling a
C style variadic function produces an error.
- Using the
va_start, va_arg, va_copy, va_end or va_list functions produces errors.
C style variadic functions has been replaced by overloading, templates and variadic template functions.
The C++ Core Guidelines identifies issues that this feature helps to mitigate.
F.55: Don’t use va_arg arguments
ES.34: Don’t define a (C-style) variadic function
No deprecated
[[static_analysis("no_deprecated")]]
|
no_deprecated is a subset of modern.
|
- Using anything that has the deprecated attribute on it produces an error.
Deprecated functionality is not modern.
Use std::array
[[static_analysis("use_std_array")]]
|
use_std_array is a subset of modern.
|
- Declaring a
C style/core C++ array variable, whether locally or in a class, produces an error.
- It is okay to use array literals when initializing
std::array and other collections.
Use std::array instead of C style/core C++ array.
What may safer and modern analyzers be composed of in the future?
No include
[[static_analysis("no_include")]]
|
no_include is a subset of modern.
|
The preprocessor directive #include has been replaced with import.
Don’t add the static analyzer until #embed is added.
NOTE: This may be impossible to implement as preprocessing occurs before compilation.
No goto
[[static_analysis("no_goto")]]
|
no_goto is a subset of modern.
|
- Using the
goto keyword produces an error.
Don’t add until break and continue to a label is added. Also a really easy to use finite state machine library may be needed.
The C++ Core Guidelines identifies issues that this feature helps to mitigate.
Use std::function_ref
[[static_analysis("use_function_ref")]]
|
use_function_ref is a subset of modern.
|
- Declaring a
C style/core C++ function pointer, whether locally or in a class, produces an error.
- Declaring a
C style/core C++ member function pointer, whether locally or in a class, produces an error.
- It is okay to use [member] function pointer literals when initializing
std::function_ref and others.
Use std::function_ref instead of C style/core C++ [member] function pointers. std::function_ref can bind to stateful and stateless, free and member functions. It saves programmers from having to include a void* state parameter in their function pointer types and it also saves from having to include void* state parameter along side the function pointer type in each function where the function pointer type is used in function declarations. Neither of which could be performed with the "use_lvalue_references" static analyzer.
NOTE:
- This can’t be performed until
nontype_t std::function_ref gets standardized.
Summary
By adding static analysis to the C++ language we can make the language safer and easier to teach because we can restrict how much of the language we use. Human readable errors and references turns the compiler into a teacher freeing human teachers to focus on what the compiler doesn’t handle.
Frequently Asked Questions
Shouldn’t these be warnings instead of errors?
NO, otherwise we’ll be stuck with what we just have. C++ compilers produces plenty of warnings. C++ static analyzers produces plenty of warnings. However, when some one talks about creating a new language, then old language syntax becomes invalid i.e. errors. This is for programmers. Programmers and businesses rarely upgrade their code unless they are forced to. Businesses and Government(s) want errors, as well, in order to ensure code quality and the assurance that bad code doesn’t exist anywhere in the module. This is also important from a language standpoint because we are essentially pruning; somewhat. Keep in mind that all of these pruned features still have use now. In the future, as more constructs are built upon these pruned features, which is why they need to be part of the language, just not a part of everyday usage of the language.
Why at the module level? Why not safe and unsafe blocks?
Programmers and businesses rarely upgrade their code unless they are forced to. New programmers need training wheels and some of us older programmers like them too. Due to the proliferation of government regulations and oversight, businesses have acquired software composition analysis services and tools. These services map security errors to specific versions of modules; specifically programming artifacts such as executables and libraries. As such, businesses want to know if a module is reasonably safe.
Any arguments provided after the name of the analyzer can be forwarded onto the analyzer.
[[static_analysis("some_future_analyzer", 1, true, 0.5, "Hello World")]]
In this case, 1, true, 0.5, "Hello World" would all be forwarded to the static analyzer “some_future_analyzer”. None of the current analyzers use this functionality so this just illustrates distant future work where we can define these analyzers in a standard fashion but that can’t happen until we have a code DOM. As such, how these arguments are forwarded are currently compiler specific.
You must really hate pointers?
Actually, I love C, C++ and pointers.
- I recognize that most of the time, when I code, that I don’t need them.
- I recognize that past fundamental
C++ libraries use pointers but the users of those libraries don’t need them.
- I recognize that present fundamental libraries such
function_ref uses void* for type erasure but the users of function_ref, most of the time, won’t need it.
- I recognize that future fundamental libraries such as dynamic polymorphic traits also need pointers for type erasure but they don’t expect their users to fidget with raw pointers.
- I also recognize that 1 programmer writes a library but hundreds use the library without needing the same parts of C++ used in its creation.
- Pointers are simple and easy for memory mapped hardware but many C++ programmers don’t operate at this level.
- A few will create an OS [driver] but thousands will use it.
The fact is pointers, unsafe casts, union, mutable and goto are the engine of C++ change. As such it would be foolish to remove them but it is also unrealistic for users/drivers of a vehicle to have to drive with nothing between them and the engine, without listening to them clamor for interior finishing.
C++ can’t standardize specific static analyzers
- Can’t
C++ provide the static_analysis attribute so that static analyzers can be called?
- Can’t
C++ reserve unscoped or names that start with std., c++., cpp., cxx. or c. are for future standardization?
- Can’t
C++ reserve the names of static analyzers in the reserved C++ static analyzer namespace?
- Can’t
C++ recommend these reserved static analyzers and leave it to the compiler writers to appease their users that clamor for them?
Do you fear that this could create a “subset of C++” that “could split the user community and cause acrimony”?
First of all, let’s consider the quotes of Bjarne Stroustrup that this question are based upon.
|
“being defined by an ‘industry consortium.’ I am not in favor of language subsets or dialects. I am especially not fond of subsets that cannot support the standard library so that the users of that subset must invent their own incompatible foundation libraries. I fear that a defined subset of C++ could split the user community and cause acrimony”
|
Does this paper create a subset? YES. Like it or not C++ already have a couple of subsets; some positive, some quasi. Freestanding is a subset for low level programming. This proposal primarily focus on high level programming but there is nothing preventing the creation of [[static_analysis("freestanding")]] which enforces freestanding. The C++ value categories has to some degree fractured the community into a clergy class that thoroughly understand its intracacies and a leity class that gleefully uses it.
Does this paper split the user community? YES and NO. It splits code into safer vs. less safe, high level vs. low level. However, this is performed at the module level, allowing the same programmer to decide what falls on either side of the fence. This would not be performed by an industry consortium but rather the standard. Safer modules can be used by less safe modules. Less safe modules can partly be used by safer modules, such as with the standard module. This latter impact is already minimalized because the standard frequently write their library code in C++ fashion instead of a C fashion.
|
“Are there any features you’d like to remove from C++?”
Not really. People who ask this kind of question usually think of one of the major features such as multiple inheritance, exceptions, templates, or run-time type identification. C++ would be incomplete without those. I have reviewed their design over the years, and together with the standards committee I have improved some of their details, but none could be removed without doing damage.
Most of the features I dislike from a language-design perspective (e.g., the declarator syntax and array decay) are part of the C subset of C++ and couldn’t be removed without doing harm to programmers working under real-world conditions. C++'s C compatibility was a key language design decision rather than a marketing gimmick. Compatibility has been difficult to achieve and maintain, but real benefits to real programmers resulted, and still result today. By now, C++ has features that allow a programmer to refrain from using the most troublesome C features. For example, standard library containers such as vector, list, map, and string can be used to avoid most tricky low-level pointer manipulation.
|
The beauty of this proposal is it does not and it does remove features from C++. Like the standard library, it allows programmers to refrain from using the most troublesome C and C++ features.
|
“Within C++, there is a much smaller and cleaner language struggling to get out”
|
Both making things smaller and cleaner requires removing something. When creating a new language, removing things happens extensively at the beginning but, frequently, features have to be added back in, when programmers clamor for them. This paper cleans up a programmers use of the C++ language, meaning less C++ has to be taught immediately, thus making things simpler. As a programmer matures, features can be gradually added to their repertoire, just as it was added to ours. After all, isn’t C++ larger now, than when we started programming in C++.
References
Jarrad J. Waterloo <descender76 at gmail dot com>
C++ is the next C++
Table of contents
Abstract
Programmer’s, Businesses and Government(s) want C++ to be safer and simpler. This has led some
C++programmers to create new programming languages or preprocessors, which again is a new language. This paper discusses using static analysis to make theC++language itself safer and simpler.Motivating Examples
Following is a wishlist. Most are optional. While, they all would be of benefit. It all starts with a new repeatable module level attribute that would preferably be applied once in the
primary module interface unitand would automatically apply to it and allmodule implementation unit(s). It could also be applied to amodule implementation unitbut that would generally be less useful. However, it might aid in gradual migration.static_analysisattribute could be passed as either an environment variable and/or command line argument to compilers so it could be used by pipelines to assert the degree of conformance to the defined static analyzer without actually changing the source.The name of the static analyzer are dotted. Unscoped or names that start with
std.,c++.,cpp.,cxx.orc.are reserved for standardization.This proposal wants to stardardize two overarching static analyzer names;
saferandmodern.The
saferanalyzer is for safety, primarily memory related. It is for those businesses and programmers who must conform to safety standards.The
saferanalyzer is a subset ofmodernanalyzer. Themodernanalyzer goes beyond just memory and safety concerns. It can be thought of as bleeding edge. It is for those businesses and programmers who commit to safety and higher quality modern code.Neither is concerned about formatting or nitpicking. Both static analyzers only produce errors. These are meant for programmers, businesses and governments in which safety takes precedence. They both represent +∞. When a new version of the standard is released and adds new sub static analyzers than everyone’s code is broken, until their code is fixed. These sub static analyzers usually consist of features that have been mostly replaced with some other feature. It would be ideal if the errors produced not only say that the code is wrong but also provide a link to html page(s) maintained by the
C++teaching group, the authors of theC++ Core Guidelines[1] and compiler specific errors. These pages should provide example(s) of what is being replaced and by what was it replaced. Mentioning the version of theC++standard would also be helpful.All modules can be used even if they don’t use the
static_analysisattribute as this allows gradual adoption.What are the
saferandmodernanalyzers composed of?These overarching static analyzers are composed of multiple static analyzers which can be used individually to allow a degree of gradual adoption.
Use lvalue references
use_lvalue_referencesis a subset ofsafer.constcharacter type” or “constpointer toconstcharacter type” and their arguments were string literals.std::stringandstd::string_viewmust be creatable at compile timeWHY?
C++has been advocated to programmers in other programming languages who complain about memory issues. This allows us to show them what we have been saying for decades.lvalue references
1985: Cfront 1.0 [2]
STL
1992 [2:1]
std::unique_ptr,std::shared_ptr,std::weak_ptr,std::reference_wrapper,std::make_sharedC++11
std::make_uniqueC++14
std::string_view,std::optional,std::any,std::variantC++17
std::make_sharedsupport arrays,std::spanC++20
The
C++ Core Guidelines[1:1] identifies issues that this feature helps to mitigate.P.4: Ideally, a program should be statically type safe[3]P.6: What cannot be checked at compile time should be checkable at run time[4]P.7: Catch run-time errors early[5]P.8: Don’t leak any resources[6]P.11: Encapsulate messy constructs, rather than spreading through the code[7]P.12: Use supporting tools as appropriate[8]P.13: Use support libraries as appropriate[9]I.4: Make interfaces precisely and strongly typed[10]I.11: Never transfer ownership by a raw pointer (T*) or reference (T&)[11]I.12: Declare a pointer that must not be null as not_null[12]I.13: Do not pass an array as a single pointer[13]I.23: Keep the number of function arguments low[14]F.7: For general use, take T* or T& arguments rather than smart pointers[15]F.15: Prefer simple and conventional ways of passing information[16]F.22: Use T* or owner<T*> to designate a single object[17]F.23: Use a not_null<T> to indicate that “null” is not a valid value[18]F.25: Use a zstring or a not_null<zstring> to designate a C-style string[19]F.26: Use a unique_ptr<T> to transfer ownership where a pointer is needed[20]F.27: Use a shared_ptr<T> to share ownership[21]F.42: Return a T* to indicate a position (only)[22]F.43: Never (directly or indirectly) return a pointer or a reference to a local object[23]C.31: All resources acquired by a class must be released by the class’s destructor[24]C.32: If a class has a raw pointer (T*) or reference (T&), consider whether it might be owning[25]C.33: If a class has an owning pointer member, define a destructor[26]C.149: Use unique_ptr or shared_ptr to avoid forgetting to delete objects created using new[27]C.150: Use make_unique() to construct objects owned by unique_ptrs[28]C.151: Use make_shared() to construct objects owned by shared_ptrs[29]R.1: Manage resources automatically using resource handles and RAII (Resource Acquisition Is Initialization)[30]R.2: In interfaces, use raw pointers to denote individual objects (only)[31]R.3: A raw pointer (a T*) is non-owning[32]R.5: Prefer scoped objects, don’t heap-allocate unnecessarily[33]R.10: Avoid malloc() and free()[34]R.11: Avoid calling new and delete explicitly[35]R.12: Immediately give the result of an explicit resource allocation to a manager object[36]R.13: Perform at most one explicit resource allocation in a single expression statement[37]R.14: Avoid [] parameters, prefer span[38]R.15: Always overload matched allocation/deallocation pairs[39]R.20: Use unique_ptr or shared_ptr to represent ownership[40]R.22: Use make_shared() to make shared_ptrs[41]R.23: Use make_unique() to make unique_ptrs[42]ES.20: Always initialize an object[43]ES.24: Use a unique_ptr<T> to hold pointers[44]ES.42: Keep use of pointers simple and straightforward[45]ES.47: Use nullptr rather than 0 or NULL[46]ES.60: Avoid new and delete outside resource management functions[47]ES.61: Delete arrays using delete[] and non-arrays using delete[48]ES.65: Don’t dereference an invalid pointer[49]E.13: Never throw while being the direct owner of an object[50]CPL.1: Prefer C++ to C[51]Gotchas
Usage of smart pointers
This static analyzer causes programmers to use 2 extra characters when using smart pointers,
->vs(*)., since the overloaded->operator returns a pointer.the main function and environment variables
A shim module is needed in order to transform main and env functions into a more C++ friendly functions. These have been asked for years.
A Modern C++ Signature for main[52]Desert Sessions: Improving hostile environment interactions[53]No unsafe casts
no_unsafe_castsis a subset ofsafer.C/core cast produces an error.reinterpret_castproduces an error.const_castproduces an error.Why?
C/core cast was replaced bystatic_castanddynamic_cast.reinterpret_castis needed more for library authors than their users. For library users it usually just causes problems and questions. It is rarely used in dailyC++when coding at a higher level.const_castis needed more for library authors than their users. It is a means for the programmer to lie to oneself. For library users it usually just causes problems and questions. It is rarely used in dailyC++when coding at a higher level.The
C++ Core Guidelines[1:2] identifies issues that this feature helps to mitigate.C.146: Use dynamic_cast where class hierarchy navigation is unavoidable[54]ES.48: Avoid casts[55]ES.49: If you must use a cast, use a named cast[56]ES.50: Don’t cast away const[57]No unions
no_unionis a subset ofsafer.unionkeyword produces an error.It was replaced by
std::variant, which is safer.The
C++ Core Guidelines[1:3] identifies issues that this feature helps to mitigate.C.181: Avoid “naked” unions[58]No mutable
no_mutableis a subset ofsafer.mutablekeyword produces an error.The programmer shall not lie to oneself. The
mutablekeyword violates the safety ofconstand is rarely used at a high level.No new or delete
no_new_deleteis a subset ofsafer.newanddeletekeywords to allocate and deallocate memory produces an error.It was replaced by
std::make_uniqueandstd::make_shared, which are safer.The
C++ Core Guidelines[1:4] identifies issues that this feature helps to mitigate.F.26: Use a unique_ptr<T> to transfer ownership where a pointer is needed[20:1]F.27: Use a shared_ptr<T> to share ownership[21:1]C.149: Use unique_ptr or shared_ptr to avoid forgetting to delete objects created using new[27:1]C.150: Use make_unique() to construct objects owned by unique_ptrs[28:1]C.151: Use make_shared() to construct objects owned by shared_ptrs[29:1]R.11: Avoid calling new and delete explicitly[35:1]R.20: Use unique_ptr or shared_ptr to represent ownership[40:1]R.22: Use make_shared() to make shared_ptrs[41:1]R.23: Use make_unique() to make unique_ptrs[42:1]ES.60: Avoid new and delete outside resource management functions[47:1]ES.61: Delete arrays using delete[] and non-arrays using delete[48:1]No volatile
no_volatileis a subset ofsafer.volatilekeyword produces an error.The
volatilekeyword has nothing to do with concurrency. Usestd::atomicorstd::mutexinstead.The
C++ Core Guidelines[1:5] identifies issues that this feature helps to mitigate.CP.8: Don’t try to use volatile for synchronization[59]No
Cstyle variadic functionsno_c_style_variadic_functionsis a subset ofsafer.Cstyle variadic function produces an error.Cstyle variadic function produces an error.va_start,va_arg,va_copy,va_endorva_listfunctions produces errors.Cstyle variadic functions has been replaced by overloading, templates and variadic template functions.The
C++ Core Guidelines[1:6] identifies issues that this feature helps to mitigate.F.55: Don’t use va_arg arguments[60]ES.34: Don’t define a (C-style) variadic function[61]No deprecated
no_deprecatedis a subset ofmodern.Deprecated functionality is not modern.
Use
std::arrayuse_std_arrayis a subset ofmodern.Cstyle/coreC++array variable, whether locally or in a class, produces an error.std::arrayand other collections.Use
std::arrayinstead ofCstyle/coreC++array.What may
saferandmodernanalyzers be composed of in the future?No include
no_includeis a subset ofmodern.The preprocessor directive
#includehas been replaced withimport.Don’t add the static analyzer until
#embedis added.NOTE: This may be impossible to implement as preprocessing occurs before compilation.
No goto
no_gotois a subset ofmodern.gotokeyword produces an error.Don’t add until
breakandcontinueto a label is added. Also a really easy to use finite state machine library may be needed.The
C++ Core Guidelines[1:7] identifies issues that this feature helps to mitigate.ES.76: Avoid goto[62]Use
std::function_refuse_function_refis a subset ofmodern.Cstyle/coreC++function pointer, whether locally or in a class, produces an error.Cstyle/coreC++member function pointer, whether locally or in a class, produces an error.std::function_refand others.Use
std::function_refinstead ofCstyle/coreC++[member] function pointers.std::function_refcan bind to stateful and stateless, free and member functions. It saves programmers from having to include avoid*state parameter in their function pointer types and it also saves from having to includevoid*state parameter along side the function pointer type in each function where the function pointer type is used in function declarations. Neither of which could be performed with the"use_lvalue_references"static analyzer.NOTE:
nontype_t[63]std::function_ref[64] gets standardized.Summary
By adding static analysis to the
C++language we can make the language safer and easier to teach because we can restrict how much of the language we use. Human readable errors and references turns the compiler into a teacher freeing human teachers to focus on what the compiler doesn’t handle.Frequently Asked Questions
Shouldn’t these be warnings instead of errors?
NO, otherwise we’ll be stuck with what we just have.
C++compilers produces plenty of warnings.C++static analyzers produces plenty of warnings. However, when some one talks about creating a new language, then old language syntax becomes invalid i.e. errors. This is for programmers. Programmers and businesses rarely upgrade their code unless they are forced to. Businesses and Government(s) want errors, as well, in order to ensure code quality and the assurance that bad code doesn’t exist anywhere in the module. This is also important from a language standpoint because we are essentially pruning; somewhat. Keep in mind that all of these pruned features still have use now. In the future, as more constructs are built upon these pruned features, which is why they need to be part of the language, just not a part of everyday usage of the language.Why at the module level? Why not safe and unsafe blocks?
Programmers and businesses rarely upgrade their code unless they are forced to. New programmers need training wheels and some of us older programmers like them too. Due to the proliferation of government regulations and oversight, businesses have acquired
software composition analysisservices and tools. These services map security errors to specific versions of modules; specifically programming artifacts such as executables and libraries. As such, businesses want to know if a module is reasonably safe.How do we configure future analyzers?
Any arguments provided after the name of the analyzer can be forwarded onto the analyzer.
In this case,
1, true, 0.5, "Hello World"would all be forwarded to the static analyzer “some_future_analyzer”. None of the current analyzers use this functionality so this just illustrates distant future work where we can define these analyzers in a standard fashion but that can’t happen until we have a code DOM. As such, how these arguments are forwarded are currently compiler specific.You must really hate pointers?
Actually, I love
C,C++and pointers.C++libraries use pointers but the users of those libraries don’t need them.function_refusesvoid*for type erasure but the users offunction_ref, most of the time, won’t need it.The fact is pointers, unsafe casts,
union,mutableandgotoare the engine of C++ change. As such it would be foolish to remove them but it is also unrealistic for users/drivers of a vehicle to have to drive with nothing between them and the engine, without listening to them clamor for interior finishing.C++ can’t standardize specific static analyzers
C++provide thestatic_analysisattribute so that static analyzers can be called?C++reserve unscoped or names that start withstd.,c++.,cpp.,cxx.orc.are for future standardization?C++reserve the names of static analyzers in the reservedC++static analyzer namespace?C++recommend these reserved static analyzers and leave it to the compiler writers to appease their users that clamor for them?Do you fear that this could create a “subset of C++” that “could split the user community and cause acrimony”? [65]
First of all, let’s consider the quotes of Bjarne Stroustrup that this question are based upon.
“being defined by an ‘industry consortium.’ I am not in favor of language subsets or dialects. I am especially not fond of subsets that cannot support the standard library so that the users of that subset must invent their own incompatible foundation libraries. I fear that a defined subset of C++ could split the user community and cause acrimony” [65:1]
Does this paper create a subset? YES. Like it or not
C++already have a couple of subsets; some positive, some quasi.Freestandingis a subset for low level programming. This proposal primarily focus on high level programming but there is nothing preventing the creation of[[static_analysis("freestanding")]]which enforcesfreestanding. TheC++value categories has to some degree fractured the community into a clergy class that thoroughly understand its intracacies and a leity class that gleefully uses it.Does this paper split the user community? YES and NO. It splits code into safer vs. less safe, high level vs. low level. However, this is performed at the module level, allowing the same programmer to decide what falls on either side of the fence. This would not be performed by an industry consortium but rather the standard. Safer modules can be used by less safe modules. Less safe modules can partly be used by safer modules, such as with the standard module. This latter impact is already minimalized because the standard frequently write their library code in
C++fashion instead of aCfashion.“Are there any features you’d like to remove from C++?” [66]
Not really. People who ask this kind of question usually think of one of the major features such as multiple inheritance, exceptions, templates, or run-time type identification. C++ would be incomplete without those. I have reviewed their design over the years, and together with the standards committee I have improved some of their details, but none could be removed without doing damage. [66:1]
Most of the features I dislike from a language-design perspective (e.g., the declarator syntax and array decay) are part of the C subset of C++ and couldn’t be removed without doing harm to programmers working under real-world conditions. C++'s C compatibility was a key language design decision rather than a marketing gimmick. Compatibility has been difficult to achieve and maintain, but real benefits to real programmers resulted, and still result today. By now, C++ has features that allow a programmer to refrain from using the most troublesome C features. For example, standard library containers such as vector, list, map, and string can be used to avoid most tricky low-level pointer manipulation. [66:2]
The beauty of this proposal is it does not and it does remove features from C++. Like the standard library, it allows programmers to refrain from using the most troublesome
CandC++features.“Within C++, there is a much smaller and cleaner language struggling to get out” [67]
Both making things smaller and cleaner requires removing something. When creating a new language, removing things happens extensively at the beginning but, frequently, features have to be added back in, when programmers clamor for them. This paper cleans up a programmers use of the
C++language, meaning lessC++has to be taught immediately, thus making things simpler. As a programmer matures, features can be gradually added to their repertoire, just as it was added to ours. After all, isn’tC++larger now, than when we started programming inC++.References
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines ↩︎ ↩︎ ↩︎ ↩︎ ↩︎ ↩︎ ↩︎ ↩︎
https://en.cppreference.com/w/cpp/language/history ↩︎ ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#p4-ideally-a-program-should-be-statically-type-safe ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#p6-what-cannot-be-checked-at-compile-time-should-be-checkable-at-run-time ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#p7-catch-run-time-errors-early ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#p8-dont-leak-any-resources ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#p11-encapsulate-messy-constructs-rather-than-spreading-through-the-code ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#p12-use-supporting-tools-as-appropriate ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#p13-use-support-libraries-as-appropriate ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#i4-make-interfaces-precisely-and-strongly-typed ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#i11-never-transfer-ownership-by-a-raw-pointer-t-or-reference-t ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#i12-declare-a-pointer-that-must-not-be-null-as-not_null ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#i13-do-not-pass-an-array-as-a-single-pointer ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#i23-keep-the-number-of-function-arguments-low ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#f7-for-general-use-take-t-or-t-arguments-rather-than-smart-pointers ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#f15-prefer-simple-and-conventional-ways-of-passing-information ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#f22-use-t-or-ownert-to-designate-a-single-object ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#f23-use-a-not_nullt-to-indicate-that-null-is-not-a-valid-value ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#f25-use-a-zstring-or-a-not_nullzstring-to-designate-a-c-style-string ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#f26-use-a-unique_ptrt-to-transfer-ownership-where-a-pointer-is-needed ↩︎ ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#f27-use-a-shared_ptrt-to-share-ownership ↩︎ ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#f42-return-a-t-to-indicate-a-position-only ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#f43-never-directly-or-indirectly-return-a-pointer-or-a-reference-to-a-local-object ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c31-all-resources-acquired-by-a-class-must-be-released-by-the-classs-destructor ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c32-if-a-class-has-a-raw-pointer-t-or-reference-t-consider-whether-it-might-be-owning ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c33-if-a-class-has-an-owning-pointer-member-define-a-destructor ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c149-use-unique_ptr-or-shared_ptr-to-avoid-forgetting-to-delete-objects-created-using-new ↩︎ ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c150-use-make_unique-to-construct-objects-owned-by-unique_ptrs ↩︎ ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c151-use-make_shared-to-construct-objects-owned-by-shared_ptrs ↩︎ ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#r1-manage-resources-automatically-using-resource-handles-and-raii-resource-acquisition-is-initialization ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#r2-in-interfaces-use-raw-pointers-to-denote-individual-objects-only ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#r3-a-raw-pointer-a-t-is-non-owning ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#r5-prefer-scoped-objects-dont-heap-allocate-unnecessarily ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#r10-avoid-malloc-and-free ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#r11-avoid-calling-new-and-delete-explicitly ↩︎ ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#r12-immediately-give-the-result-of-an-explicit-resource-allocation-to-a-manager-object ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#r13-perform-at-most-one-explicit-resource-allocation-in-a-single-expression-statement ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#r14-avoid--parameters-prefer-span ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#r15-always-overload-matched-allocationdeallocation-pairs ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#r20-use-unique_ptr-or-shared_ptr-to-represent-ownership ↩︎ ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#r22-use-make_shared-to-make-shared_ptrs ↩︎ ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#r23-use-make_unique-to-make-unique_ptrs ↩︎ ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es20-always-initialize-an-object ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es24-use-a-unique_ptrt-to-hold-pointers ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es42-keep-use-of-pointers-simple-and-straightforward ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es47-use-nullptr-rather-than-0-or-null ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es60-avoid-new-and-delete-outside-resource-management-functions ↩︎ ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es61-delete-arrays-using-delete-and-non-arrays-using-delete ↩︎ ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es65-dont-dereference-an-invalid-pointer ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#e13-never-throw-while-being-the-direct-owner-of-an-object ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#cpl1-prefer-c-to-c ↩︎
https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2017/p0781r0.html ↩︎
https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2018/p1275r0.html ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c146-use-dynamic_cast-where-class-hierarchy-navigation-is-unavoidable ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es48-avoid-casts ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es49-if-you-must-use-a-cast-use-a-named-cast ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es50-dont-cast-away-const ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c181-avoid-naked-unions ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#cp8-dont-try-to-use-volatile-for-synchronization ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#f55-dont-use-va_arg-arguments ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#-es34-dont-define-a-c-style-variadic-function ↩︎
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es76-avoid-goto ↩︎
https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2022/p2472r3.html ↩︎
https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2022/p0792r11.html ↩︎
https://www.stroustrup.com/bs_faq.html#EC++ ↩︎ ↩︎
https://www.stroustrup.com/bs_faq.html#remove-from-C++ ↩︎ ↩︎ ↩︎
https://www.stroustrup.com/quotes.html ↩︎