From 9045f528b9b6f21f0d15475d2dffd49bb51598fc Mon Sep 17 00:00:00 2001
From: zhangmeng <775834166@qq.com>
Date: 星期一, 26 十二月 2022 09:16:03 +0800
Subject: [PATCH] optimize cpu

---
 src/interface_bus_api.cpp |   36 +++++++++++++++++++++++-------------
 1 files changed, 23 insertions(+), 13 deletions(-)

diff --git a/src/interface_bus_api.cpp b/src/interface_bus_api.cpp
index 638f31a..9d110c7 100644
--- a/src/interface_bus_api.cpp
+++ b/src/interface_bus_api.cpp
@@ -31,6 +31,7 @@
         bus *b = new bus;
         bhome_msg::ProcInfo pi;
         if (pi.ParseFromArray(proc_info, proc_info_len)) get<8>(*b) = pi.proc_id();
+        std::move(pi);
 
         const auto& url_hb = get_url(URLHeartBeat);
         respond_survey(url_hb,string{(char*)proc_info,(size_t)proc_info_len},&get<5>(*b));
@@ -106,8 +107,11 @@
     for(int i = 0; i < mtl.topic_list_size(); i++){
         mtl2.add_topic_list(mtl.topic_list(i));
     }
+    std::move(mtl);
 
     string msg(mtl2.SerializeAsString());
+    std::move(mtl2);
+
     const auto& url = get_url(URLRegTopic);
     if (url.empty()) {
         set_last_error("bus_register_topics url empty");
@@ -135,7 +139,7 @@
         return false;
     }
 
-    auto url(get_url(URLQueryTopic));
+    const auto& url = get_url(URLQueryTopic);
     if (url.empty()) {
         set_last_error("bus_query_topic_address url empty");
         return false;
@@ -173,7 +177,7 @@
         return false;
     }
 
-    auto url(get_url(URLQueryProcs));
+    const auto& url = get_url(URLQueryProcs);
     if (url.empty()) {
         set_last_error("bus_query_procs url empty");
         return false;
@@ -221,6 +225,7 @@
     for(int i = 0; i < mtl.topic_list_size(); i ++){
         subscribe_topic(mtl.topic_list(i), &get<2>(*b));
     }
+    std::move(mtl);
 
 ///////////////////////////////////////////////////
     auto ret = simple_request(url, topics, topics_len, reply, reply_len, timeout_ms);
@@ -283,6 +288,8 @@
     newPub.set_data(string{(const char*)msgpub, (const size_t)msgpub_len});
 
     string msg(newPub.SerializeAsString());
+    std::move(newPub);
+
     auto ret = publish(pub.topic(), msg.data(), msg.size(), &get<1>(*b));
     if (ret > 0) return true;
     return false;
@@ -341,11 +348,7 @@
     string url{};
     // BHQueryTopicAddress鑾峰彇proc_id
     string procid{};
-    if (get_proc_id_from_MsgRequestTopic(request, request_len, timeout_ms, &procid)){
-        // PRNTVITAG("get_proc_id_from_MsgRequestTopic failed");
-        // return false;
-        url = "ipc:///tmp/" + procid;
-    }
+
     if (remote && remote_len > 0){
         BHAddress addr;
         if (addr.ParseFromArray(remote, remote_len)){
@@ -354,7 +357,12 @@
                 // printf("======>> BHRequest use remote address %s\n", url.c_str());
             }
         }
+    } else if (get_proc_id_from_MsgRequestTopic(request, request_len, timeout_ms, &procid)){
+        // PRNTVITAG("get_proc_id_from_MsgRequestTopic failed");
+        // return false;
+        url = "ipc:///tmp/" + procid;
     }
+
     if (url.empty()) {
         set_last_error("bus_request url empty");
         return false;
@@ -383,13 +391,15 @@
     string msg;
     auto ret = read_request(src, &msg, timeout_ms, &get<6>(*b));
     // printf("bus_recv_request read_request ret %d msg %s\n", ret, msg.c_str());
-    if (ret != 0) return false;
+    if (ret < 0) return false;
 
-    string procid{};
-    if (get_proc_id_from_MsgRequestTopic(msg.data(), msg.size(), timeout_ms, &procid)){
-        // PRNTVITAG("get_proc_id_from_MsgRequestTopic failed");
-        // return false;
-        copy_memory(proc_id, proc_id_len, procid.data(), procid.size());
+    if (ret == REPLY_IPC){
+        string procid{};
+        if (get_proc_id_from_MsgRequestTopic(msg.data(), msg.size(), timeout_ms, &procid)){
+            // PRNTVITAG("get_proc_id_from_MsgRequestTopic failed");
+            // return false;
+            copy_memory(proc_id, proc_id_len, procid.data(), procid.size());
+        }
     }
 
     copy_memory(request, request_len, msg.data(), msg.size());

--
Gitblit v1.8.0