commit a385bf151cc00b20cf8f2c361a1b53b4b0c1f08f
author Wyatt Hepler <hepler@google.com> Wed Nov 16 22:34:51 2022 +0000
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Nov 16 22:34:51 2022 +0000
tree 3fd50d4cb5fcff0a7b5190f7f06d7856739b31f7
parent fcce22cbd06b6740c87222c4614d2d4ab2a2e551

rpc: Rename channels list to avoid GCC bug

arm-none-eabi-gcc incorrectly flags a pw::rpc::Endpoint constructor
parameter as shadowing a global declaration when that constructor is
turned into a template. Rename the "channels" list to "rpc_channels" to
work around this.

Change-Id: I9b0fbdc1f68a1a2d135602c81d65a217aa7721c0
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/experimental/+/119733
Reviewed-by: Alexei Frolov <frolv@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed.google.com.iam.gserviceaccount.com>
Pigweed-Auto-Submit: Wyatt Hepler <hepler@google.com>

applications/rpc/main.cc

diff --git a/applications/rpc/main.cc b/applications/rpc/main.cc
index c53ba64..b55553c 100644
--- a/applications/rpc/main.cc
+++ b/applications/rpc/main.cc
@@ -63,11 +63,11 @@
 
 // A pw::rpc::Server can have multiple channels (e.g. a UART and a BLE
 // connection). In this case, there is only one (HDLC over UART).
-pw::rpc::Channel channels[] = {
+pw::rpc::Channel rpc_channels[] = {
     pw::rpc::Channel::Create<1>(&hdlc_channel_output)};
 
 // Declare the pw_rpc server with the HDLC channel.
-pw::rpc::Server server(channels);
+pw::rpc::Server server(rpc_channels);
 
 // Declare a buffer for decoding incoming HDLC frames.
 std::array<std::byte, kMaxTransmissionUnit> input_buffer;