View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0029006 | Community | OCCT:Coding | public | 2017-08-16 17:44 | 2018-10-03 18:25 |
Reporter | Vico Liang | Assigned To | kgv | ||
Priority | low | Severity | feature | ||
Status | closed | Resolution | no change required | ||
Target Version | 7.4.0 | ||||
Summary | 0029006: Coding Rules - define Standard_NULL to replace NULL | ||||
Description | Why not define Standard_NULL macro to present a null pointer? it can be defined as nullptr for new version C++ 11 and 0 for old version. | ||||
Tags | No tags attached. | ||||
Test case number | |||||
|
Dear colleagues, please share your opinion about this feature. |
|
What benefits will it bring? |
|
I deem 0 can represent NULL pointer pretty well. I understand that there is some rationale behind nullptr, but prefer not to define our own alternative. Let's wait until we abandon support of VS 2010 and then we can use standard nullptr. |
|
nullptr is supported in most modern C++ compiler. No objection to use standard nullptr. |
|
Yep, nullptr is supported by VS 2010, sorry for mistake. Thus there is no issue - we can use nullptr. I am closing the issue. |
Date Modified | Username | Field | Change |
---|---|---|---|
2017-08-16 17:44 | Vico Liang | New Issue | |
2017-08-16 17:44 | Vico Liang | Assigned To | => kgv |
2017-08-17 13:38 | kgv | Priority | normal => low |
2017-08-17 13:38 | kgv | Severity | minor => feature |
2017-08-17 13:39 | kgv | Summary | Define Standard_NULL to replace NULL => Coding Rules - define Standard_NULL to replace NULL |
2018-10-01 08:30 | kgv | Assigned To | kgv => abv |
2018-10-01 08:30 | kgv | Status | new => feedback |
2018-10-01 12:51 | kgv | Note Added: 0079557 | |
2018-10-02 09:52 |
|
Note Added: 0079571 | |
2018-10-02 15:16 |
|
Note Added: 0079600 | |
2018-10-02 15:16 |
|
Assigned To | abv => Vico Liang |
2018-10-02 15:16 |
|
Note Edited: 0079600 | |
2018-10-03 17:21 | Vico Liang | Note Added: 0079649 | |
2018-10-03 17:23 | Vico Liang | Assigned To | Vico Liang => kgv |
2018-10-03 17:23 | Vico Liang | Status | feedback => acknowledged |
2018-10-03 18:25 |
|
Note Added: 0079653 | |
2018-10-03 18:25 |
|
Status | acknowledged => closed |
2018-10-03 18:25 |
|
Resolution | open => no change required |