[omniORB-dev] omniORB: Remove unnecessary null pointer checks

[Markus Elfring]

Hello,

An extra null pointer check is not needed in functions like the following.
https://isocpp.org/wiki/faq/freestore-mgmt#delete-handles-null

* Destructor for omni_thread
  https://sourceforge.net/p/omniorb/svn/6666/tree/branches/4_4/omniORB/src/lib/omnithread/posix.cc#l535

* Destructor for omniServerRequest
  https://sourceforge.net/p/omniorb/svn/6666/tree/branches/4_4/omniORB/src/lib/omniORB/dynamic/serverRequest.cc#l57


Would you become interested to use a development tool like “clang-tidy”
for corresponding source code adjustments?
https://clang.llvm.org/extra/clang-tidy/checks/readability/delete-null-pointer.html

Regards,
Markus