View Issue Details

IDProjectCategoryView StatusLast Update
0029006CommunityOCCT:Codingpublic2018-10-03 18:25
ReporterVico Liang Assigned Tokgv  
PrioritylowSeverityfeature 
Status closedResolutionno change required 
Target Version7.4.0 
Summary0029006: Coding Rules - define Standard_NULL to replace NULL
DescriptionWhy 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.
TagsNo tags attached.
Test case number

Activities

kgv

2018-10-01 12:51

developer   ~0079557

Dear colleagues,

please share your opinion about this feature.

msv

2018-10-02 09:52

developer   ~0079571

What benefits will it bring?

abv

2018-10-02 15:16

manager   ~0079600

Last edited: 2018-10-02 15:16

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.

Vico Liang

2018-10-03 17:21

developer   ~0079649

nullptr is supported in most modern C++ compiler. No objection to use standard nullptr.

abv

2018-10-03 18:25

manager   ~0079653

Yep, nullptr is supported by VS 2010, sorry for mistake. Thus there is no issue - we can use nullptr.

I am closing the issue.

Issue History

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 msv Note Added: 0079571
2018-10-02 15:16 abv Note Added: 0079600
2018-10-02 15:16 abv Assigned To abv => Vico Liang
2018-10-02 15:16 abv 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 abv Note Added: 0079653
2018-10-03 18:25 abv Status acknowledged => closed
2018-10-03 18:25 abv Resolution open => no change required