(trafficserver) branch master updated: cleanup: Change legacy function name to actually reflect the logic. (#11655)

[dmeden]

This is an automated email from the ASF dual-hosted git repository.

dmeden pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 4f347919be cleanup: Change legacy function name to actually reflect 
the logic. (#11655)
4f347919be is described below

commit 4f347919bed1d9c94f4aa5653e315a14cddcb3ce
Author: Damian Meden <dme...@apache.org>
AuthorDate: Tue Aug 6 21:57:05 2024 +0200

    cleanup: Change legacy function name to actually reflect the logic. (#11655)
    
    After removing process manager the function name was never changed.
---
 src/proxy/logging/LogStandalone.cc   | 8 ++++----
 src/traffic_server/traffic_server.cc | 8 ++++----
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/proxy/logging/LogStandalone.cc 
b/src/proxy/logging/LogStandalone.cc
index 46df818915..73d64b1d2f 100644
--- a/src/proxy/logging/LogStandalone.cc
+++ b/src/proxy/logging/LogStandalone.cc
@@ -75,11 +75,11 @@ init_system(bool notify_syslog)
 }
 
 /*-------------------------------------------------------------------------
-  initialize_process_manager
+  initialize_records
   -------------------------------------------------------------------------*/
 
 static void
-initialize_process_manager()
+initialize_records()
 {
   // diags should have been initialized by caller, e.g.: sac.cc
   ink_assert(diags());
@@ -167,7 +167,7 @@ init_log_standalone(const char *pgm_name, bool one_copy)
   openlog(pgm_name, LOG_PID | LOG_NDELAY | LOG_NOWAIT, LOG_DAEMON);
 
   init_system(true);
-  initialize_process_manager();
+  initialize_records();
   diagsConfig = new DiagsConfig(pgm_name, logfile, error_tags, action_tags);
 }
 
@@ -178,7 +178,7 @@ init_log_standalone(const char *pgm_name, bool one_copy)
   simple standalone applications that do not read the records.yaml file
   and that do not need a process manager, thus it:
 
-  1) does not call initialize_process_manager
+  1) does not call initialize_records
   2) initializes the diags with use_records = false
   3) assumes multiple copies of the application can run, so does not
      do lock checking
diff --git a/src/traffic_server/traffic_server.cc 
b/src/traffic_server/traffic_server.cc
index 044889eef4..bad3b839cd 100644
--- a/src/traffic_server/traffic_server.cc
+++ b/src/traffic_server/traffic_server.cc
@@ -669,10 +669,10 @@ check_config_directories()
 }
 
 //
-// Startup process manager
+// Initialize records related features.
 //
 static void
-initialize_process_manager()
+initialize_records()
 {
   RecProcessInit(diags());
   LibRecordsConfigInit();
@@ -1847,8 +1847,8 @@ main(int /* argc ATS_UNUSED */, const char **argv)
   // fprintf's before those calls
   bind_outputs(bind_stdout, bind_stderr);
 
-  // Local process manager
-  initialize_process_manager();
+  // Records init
+  initialize_records();
 
   // Initialize file manager for TS.
   initialize_file_manager();