commit 00aa1014d39b1f7869e392ea630fb7598c282d07
author Andrei Litvin <andy314@gmail.com> Mon Jun 03 10:03:25 2024 -0400
committer GitHub <noreply@github.com> Mon Jun 03 10:03:25 2024 -0400
tree 2bf28d70dc0f994c5a1a276359f309ba1ead2ae0
parent 10d7d9af4588f5519eacd6a68376093145b4322a

Make BLE UUID string constants lowercase (#33692)

* make BLE UUID constants be lowercase, add note about bluez

* Restyle

---------

Co-authored-by: Andrei Litvin <andreilitvin@google.com>

src/ble/BleUUID.h

diff --git a/src/ble/BleUUID.h b/src/ble/BleUUID.h
index 6470cfe..37e2fdc 100644
--- a/src/ble/BleUUID.h
+++ b/src/ble/BleUUID.h
@@ -95,16 +95,22 @@
     }();
 
 // UUID of CHIP BLE service.
+// NOTE: lower-case string seems to be required at least by bluez when
+//       executing g_variant_lookup_value
+//
+//       BlueZ API https://git.kernel.org/pub/scm/bluetooth/bluez.git/tree/doc/org.bluez.Device.rst
+//       describes ServiceData as "Keys are the UUIDs in string format" however no description
+//       on actual case required
 inline constexpr char CHIP_BLE_DESC_SHORT_UUID_STR[]    = "2902";
-inline constexpr char CHIP_BLE_SERVICE_SHORT_UUID_STR[] = "FFF6";
-inline constexpr char CHIP_BLE_SERVICE_LONG_UUID_STR[]  = "0000FFF6-0000-1000-8000-00805F9B34FB";
-inline constexpr char CHIP_BLE_CHAR_1_UUID_STR[]        = "18EE2EF5-263D-4559-959F-4F9C429F9D11";
-inline constexpr char CHIP_BLE_CHAR_2_UUID_STR[]        = "18EE2EF5-263D-4559-959F-4F9C429F9D12";
-inline constexpr char CHIP_BLE_CHAR_3_UUID_STR[]        = "64630238-8772-45F2-B87D-748A83218F04";
-inline constexpr ChipBleUUID CHIP_BLE_SVC_ID            = StringToUUIDConstexpr("0000FFF6-0000-1000-8000-00805F9B34FB");
-inline constexpr ChipBleUUID CHIP_BLE_CHAR_1_UUID       = StringToUUIDConstexpr("18EE2EF5-263D-4559-959F-4F9C429F9D11");
-inline constexpr ChipBleUUID CHIP_BLE_CHAR_2_UUID       = StringToUUIDConstexpr("18EE2EF5-263D-4559-959F-4F9C429F9D12");
-inline constexpr ChipBleUUID CHIP_BLE_CHAR_3_UUID       = StringToUUIDConstexpr("64630238-8772-45F2-B87D-748A83218F04");
+inline constexpr char CHIP_BLE_SERVICE_SHORT_UUID_STR[] = "fff6";
+inline constexpr char CHIP_BLE_SERVICE_LONG_UUID_STR[]  = "0000fff6-0000-1000-8000-00805f9b34fb";
+inline constexpr char CHIP_BLE_CHAR_1_UUID_STR[]        = "18ee2ef5-263d-4559-959f-4f9c429f9d11";
+inline constexpr char CHIP_BLE_CHAR_2_UUID_STR[]        = "18ee2ef5-263d-4559-959f-4f9c429f9d12";
+inline constexpr char CHIP_BLE_CHAR_3_UUID_STR[]        = "64630238-8772-45f2-b87d-748a83218f04";
+inline constexpr ChipBleUUID CHIP_BLE_SVC_ID            = StringToUUIDConstexpr("0000fff6-0000-1000-8000-00805f9b34fb");
+inline constexpr ChipBleUUID CHIP_BLE_CHAR_1_UUID       = StringToUUIDConstexpr("18ee2ef5-263d-4559-959f-4f9c429f9d11");
+inline constexpr ChipBleUUID CHIP_BLE_CHAR_2_UUID       = StringToUUIDConstexpr("18ee2ef5-263d-4559-959f-4f9c429f9d12");
+inline constexpr ChipBleUUID CHIP_BLE_CHAR_3_UUID       = StringToUUIDConstexpr("64630238-8772-45f2-b87d-748a83218f04");
 
 } /* namespace Ble */
 } /* namespace chip */