[An extra null pointer check is not needed in functions](https://isocpp.org/wiki/faq/freestore-mgmt#delete-handles-null "Do I need to check for null before delete p?") like the following. * [DOS_File](https://github.com/papyrussolution/OpenPapyrus/blob/bbd0b6e20cd81eec7f52cb91d7780a2908e4d0be/Src/OSF/vDOS/include/dos_system.h#L32 "Destructor for DOS_File") * [LazyDupCursor](https://github.com/papyrussolution/OpenPapyrus/blob/b3614950a76445c32da5812596e51be9dafb4365/Src/OSF/BDB-6232/lang/cxx/stl/dbstl_dbc.h#L100 "Destructor for LazyDupCursor") * [THelpTopic](https://github.com/papyrussolution/OpenPapyrus/blob/bbd0b6e20cd81eec7f52cb91d7780a2908e4d0be/Src/PPLib/HELPBASE.CPP#L30 "Destructor for THelpTopic")
Activity
papyrussolution commentedon Aug 22, 2018
Thank you. We'll correct these points.
elfring commentedon Aug 22, 2018
It seems that the function “PPThreadLocalArea::SetupPhoneServiceEventResponder” is another update candidate according to an other source code transformation pattern.