[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. - [GianoAuthenticator](https://github.com/brpc/brpc/blob/30992d3a50d1d39d58f804a6311a726ad99dba37/src/brpc/policy/giano_authenticator.cpp#L41 "Destructor for GianoAuthenticator") - [SocketMap](https://github.com/brpc/brpc/blob/30992d3a50d1d39d58f804a6311a726ad99dba37/src/brpc/socket_map.cpp#L134 "Destructor for SocketMap")