Gitiles
Code Review Sign In
gerrit.openbmc.org / openbmc / phosphor-net-ipmid / a82e4d3e4dfa452f9752c6bc6e6aa1c5a7c3d247
commita82e4d3e4dfa452f9752c6bc6e6aa1c5a7c3d247[log] [tgz]
authorLei YU <yulei.sh@bytedance.com>Wed Jul 13 10:03:20 2022 +0800
committerLei YU <yulei.sh@bytedance.com>Wed Jul 13 19:55:17 2022 +0800
tree68dd158c46617b84dbacfb49dcacf1872e4bee0a
parentce1f4fc56c364b61e54b3ace0730fbce7cf89258 [diff]
message_parsers: Use const reference where possible

The code was passing std::shared_ptr<xxx> by value (or const value),
which creates unnecessary copies of shared_ptr.

Use `const std::shared_ptr<xxx>&` where possible to eliminate such
copies.

Tested: No code logic changes. Verify the build passes and the net-ipmid
        is working as before.

Signed-off-by: Lei YU <yulei.sh@bytedance.com>
Change-Id: Iad6c3b209ab8299fa477784e86c22be19f98a344
2 files changed
tree: 68dd158c46617b84dbacfb49dcacf1872e4bee0a
  1. command/
  2. sol/
  3. test/
  4. .clang-format
  5. .gitignore
  6. auth_algo.cpp
  7. auth_algo.hpp
  8. bootstrap.sh
  9. comm_module.cpp
  10. comm_module.hpp
  11. command_table.cpp
  12. command_table.hpp
  13. configure.ac
  14. crypt_algo.cpp
  15. crypt_algo.hpp
  16. endian.hpp
  17. integrity_algo.cpp
  18. integrity_algo.hpp
  19. LICENSE
  20. main.cpp
  21. main.hpp
  22. MAINTAINERS
  23. Makefile.am
  24. message.hpp
  25. message_handler.cpp
  26. message_handler.hpp
  27. message_parsers.cpp
  28. message_parsers.hpp
  29. OWNERS
  30. phosphor-ipmi-net@.service
  31. phosphor-ipmi-net@.socket
  32. prng.hpp
  33. README.md
  34. rmcp.hpp
  35. sd_event_loop.cpp
  36. sd_event_loop.hpp
  37. session.hpp
  38. sessions_manager.cpp
  39. sessions_manager.hpp
  40. socket_channel.hpp
  41. sol_module.cpp
  42. sol_module.hpp
README.md

To Build

To build this package, do the following steps:

    1. ./bootstrap.sh
    2. ./configure ${CONFIGURE_FLAGS}
    3. make

To clean the repository run `./bootstrap.sh clean`.