From dc86ace85e437ecb8a2e728e4dce36d02bbb8a6e Mon Sep 17 00:00:00 2001 From: lichao <lichao@aiotlink.com> Date: 星期五, 23 四月 2021 12:59:50 +0800 Subject: [PATCH] move ref count into msg meta, only 1 poinetr now. --- src/msg.cpp | 64 +++++++++++++------------------ 1 files changed, 27 insertions(+), 37 deletions(-) diff --git a/src/msg.cpp b/src/msg.cpp index 06b817e..ba844da 100644 --- a/src/msg.cpp +++ b/src/msg.cpp @@ -26,11 +26,28 @@ //*/ const uint32_t kMsgTag = 0xf1e2d3c4; +void *MsgI::Alloc(SharedMemory &shm, const size_t size) +{ + void *p = shm.Alloc(sizeof(Meta) + size); + if (p) { + auto pmeta = new (p) Meta; + p = pmeta + 1; + } + return p; +} +void MsgI::Free(SharedMemory &shm) +{ + assert(valid()); + shm.Dealloc(meta()); + ptr_ = nullptr; + assert(!valid()); +} + void *MsgI::Pack(SharedMemory &shm, const uint32_t head_len, const ToArray &headToArray, const uint32_t body_len, const ToArray &bodyToArray) { - void *addr = shm.Alloc(sizeof(head_len) + head_len + sizeof(body_len) + body_len); + void *addr = Alloc(shm, sizeof(head_len) + head_len + sizeof(body_len) + body_len); if (addr) { auto p = static_cast<char *>(addr); auto Pack1 = [&p](auto len, auto &writer) { @@ -47,26 +64,11 @@ bool MsgI::ParseHead(BHMsgHead &head) const { - auto p = static_cast<char *>(ptr_.get()); + auto p = get<char>(); assert(p); uint32_t msg_size = Get32(p); p += 4; return head.ParseFromArray(p, msg_size); -} - -// with ref count; -bool MsgI::MakeRC(SharedMemory &shm, void *p) -{ - if (!p) { - return false; - } - RefCount *rc = shm.New<RefCount>(); - if (!rc) { - shm.Dealloc(p); - return false; - } - MsgI(p, rc).swap(*this); - return true; } bool MsgI::Make(SharedMemory &shm, void *p) @@ -74,32 +76,20 @@ if (!p) { return false; } - MsgI(p, 0).swap(*this); + MsgI(p).swap(*this); return true; -} - -bool MsgI::EnableRefCount(SharedMemory &shm) -{ - if (!IsCounted()) { - count_ = shm.New<RefCount>(); - } - return IsCounted(); } int MsgI::Release(SharedMemory &shm) { - if (IsCounted()) { - const int n = count_->Dec(); - if (n != 0) { - return n; - } + if (!valid()) { + return 0; } - // free data - shm.Dealloc(ptr_); - ptr_ = 0; - shm.Delete(count_); - count_ = 0; - return 0; + auto n = meta()->count_.Dec(); + if (n == 0) { + Free(shm); + } + return n; } } // namespace bhome_msg -- Gitblit v1.8.0