Replace CHECK_RETURN_IF_FALSE with RET_CHECK

[pudelkoM]

This PR replaces the home-grown CHECK_RETURN_IF_FALSE macro with the more common RET_CHECK one. Semantics and behavior stay the same, no user-code changes needed. Code readability is improved a little, as the new macro is shorter, leading to less line splits.
We hope that this will make migration to absl::Status easier eventually, as we will probably go with whatever solution Google comes up with.

Fixes #579

[codecov]

Codecov Report

Merging #902 (47a694e) into main (4f0133d) will increase coverage by 0.07%.
The diff coverage is 74.74%.

@@            Coverage Diff             @@
##             main     #902      +/-   ##
==========================================
+ Coverage   79.10%   79.17%   +0.07%     
==========================================
  Files         339      339              
  Lines       30938    30915      -23     
==========================================
+ Hits        24474    24478       +4     
+ Misses       6464     6437      -27     

Impacted Files	Coverage Δ
stratum/hal/lib/barefoot/bfrt_switch.cc	57.81% <0.00%> (ø)
stratum/hal/lib/bcm/bcm_switch.cc	60.28% <0.00%> (ø)
stratum/hal/lib/common/admin_utils.cc	0.00% <0.00%> (ø)
stratum/hal/lib/np4intel/np4_chassis_manager.cc	42.36% <0.00%> (ø)
stratum/hal/lib/p4/p4_info_manager.cc	81.94% <0.00%> (ø)
stratum/hal/lib/phal/phal.cc	21.50% <0.00%> (+0.45%)	⬆️
stratum/hal/lib/phal/tai/taish_client.cc	1.02% <0.00%> (+<0.01%)	⬆️
stratum/lib/macros.h	94.44% <ø> (ø)
stratum/lib/security/auth_policy_checker.cc	31.03% <ø> (ø)
stratum/hal/lib/phal/onlp/onlp_wrapper.cc	32.16% <3.57%> (ø)
... and 51 more

[pudelkoM]

Lint failure is expected, as bfrt_table_manager_test.cc and bfrt_table_manager.cc are unformatted.