src: avoid duplicate Before/AtExitCallback structs

Currently, BeforeExitCallback and AtExitCallback are identical apart for the name of the struct. This commit introduces an ExitCallback struct with can be used in both cases to avoid the duplication. PR-URL: #19226 Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: danbev

src/env.cc

@@ -218,25 +218,25 @@ void Environment::PrintSyncTrace() const{
 }
 
 voidEnvironment::RunBeforeExitCallbacks(){
-for (BeforeExitCallback before_exit : before_exit_functions_){
+for (ExitCallback before_exit : before_exit_functions_){
  before_exit.cb_(before_exit.arg_);
  }
  before_exit_functions_.clear();
 }
 
 voidEnvironment::BeforeExit(void (*cb)(void* arg), void* arg){
- before_exit_functions_.push_back(BeforeExitCallback{cb, arg});
+ before_exit_functions_.push_back(ExitCallback{cb, arg});
 }
 
 voidEnvironment::RunAtExitCallbacks(){
-for (AtExitCallback at_exit : at_exit_functions_){
+for (ExitCallback at_exit : at_exit_functions_){
  at_exit.cb_(at_exit.arg_);
  }
  at_exit_functions_.clear();
 }
 
 voidEnvironment::AtExit(void (*cb)(void* arg), void* arg){
- at_exit_functions_.push_back(AtExitCallback{cb, arg});
+ at_exit_functions_.push_back(ExitCallback{cb, arg});
 }
 
 voidEnvironment::AddPromiseHook(promise_hook_func fn, void* arg){

src/env.h

@@ -799,17 +799,13 @@ class Environment{
 
 double* fs_stats_field_array_;
 
-structBeforeExitCallback{
+structExitCallback{
 void (*cb_)(void* arg);
 void* arg_;
  };
- std::list<BeforeExitCallback> before_exit_functions_;
+ std::list<ExitCallback> before_exit_functions_;
 
-structAtExitCallback{
-void (*cb_)(void* arg);
-void* arg_;
- };
- std::list<AtExitCallback> at_exit_functions_;
+ std::list<ExitCallback> at_exit_functions_;
 
 structPromiseHookCallback{
  promise_hook_func cb_;