Add optimized device names API and update all examples

Co-authored-by: wysaid <1430725+wysaid@users.noreply.github.com>

Author: Copilot

examples/desktop/0-print_camera_c.c

@@ -39,17 +39,13 @@ int main() {
     }
 
     // Find available devices
-    char** deviceNames;
-    size_t deviceCount;
-    if (ccap_provider_find_device_names(provider, &deviceNames, &deviceCount)) {
-        printf("Found %zu camera device(s):\n", deviceCount);
-        for (size_t i = 0; i < deviceCount; i++) {
-            printf("  %zu: %s\n", i, deviceNames[i]);
+    CcapDeviceNamesList deviceList;
+    if (ccap_provider_find_device_names_list(provider, &deviceList)) {
+        printf("Found %zu camera device(s):\n", deviceList.deviceCount);
+        for (size_t i = 0; i < deviceList.deviceCount; i++) {
+            printf("  %zu: %s\n", i, deviceList.deviceNames[i]);
         }
         printf("\n");
-
-        // Free device names
-        ccap_provider_free_device_names(deviceNames, deviceCount);
     } else {
         printf("Failed to enumerate devices\n");
     }

examples/desktop/3-capture_callback_c.c

@@ -99,13 +99,11 @@ int main(int argc, char** argv) {
     }
 
     // Find and print available devices
-    char** deviceNames;
-    size_t deviceCount;
-    if (ccap_provider_find_device_names(provider, &deviceNames, &deviceCount)) {
-        for (size_t i = 0; i < deviceCount; i++) {
-            printf("## Found video capture device: %s\n", deviceNames[i]);
+    CcapDeviceNamesList deviceList;
+    if (ccap_provider_find_device_names_list(provider, &deviceList)) {
+        for (size_t i = 0; i < deviceList.deviceCount; i++) {
+            printf("## Found video capture device: %s\n", deviceList.deviceNames[i]);
         }
-        ccap_provider_free_device_names(deviceNames, deviceCount);
     }
 
     // Set camera properties

examples/desktop/4-example_with_glfw_c.c

@@ -94,13 +94,11 @@ int main(int argc, char** argv) {
     }
 
     // Find and print available devices
-    char** deviceNames;
-    size_t deviceCount;
-    if (ccap_provider_find_device_names(provider, &deviceNames, &deviceCount)) {
-        for (size_t i = 0; i < deviceCount; i++) {
-            printf("## Found video capture device: %s\n", deviceNames[i]);
+    CcapDeviceNamesList deviceList;
+    if (ccap_provider_find_device_names_list(provider, &deviceList)) {
+        for (size_t i = 0; i < deviceList.deviceCount; i++) {
+            printf("## Found video capture device: %s\n", deviceList.deviceNames[i]);
         }
-        ccap_provider_free_device_names(deviceNames, deviceCount);
     }
 
     // Set camera properties

examples/desktop/utils/helper.cpp

@@ -37,13 +37,12 @@ int getCurrentWorkingDirectory(char* buffer, int size) {
 }
 
 int selectCamera(CcapProvider* provider) {
-    char** deviceNames;
-    size_t deviceCount;
+    CcapDeviceNamesList deviceList;
 
-    if (ccap_provider_find_device_names(provider, &deviceNames, &deviceCount) && deviceCount > 1) {
+    if (ccap_provider_find_device_names_list(provider, &deviceList) && deviceList.deviceCount > 1) {
         printf("Multiple devices found, please select one:\n");
-        for (size_t i = 0; i < deviceCount; i++) {
-            printf("  %zu: %s\n", i, deviceNames[i]);
+        for (size_t i = 0; i < deviceList.deviceCount; i++) {
+            printf("  %zu: %s\n", i, deviceList.deviceNames[i]);
         }
 
         int selectedIndex;
@@ -52,21 +51,16 @@ int selectCamera(CcapProvider* provider) {
             selectedIndex = 0;
         }
 
-        if (selectedIndex < 0 || selectedIndex >= static_cast<int>(deviceCount)) {
+        if (selectedIndex < 0 || selectedIndex >= static_cast<int>(deviceList.deviceCount)) {
             selectedIndex = 0;
-            fprintf(stderr, "Invalid index, using the first device: %s\n", deviceNames[0]);
+            fprintf(stderr, "Invalid index, using the first device: %s\n", deviceList.deviceNames[0]);
         } else {
-            printf("Using device: %s\n", deviceNames[selectedIndex]);
+            printf("Using device: %s\n", deviceList.deviceNames[selectedIndex]);
         }
 
-        ccap_provider_free_device_names(deviceNames, deviceCount);
         return selectedIndex;
     }
 
-    if (deviceCount > 0) {
-        ccap_provider_free_device_names(deviceNames, deviceCount);
-    }
-
     return -1; // One or no device, use default.
 }
 

include/ccap_c.h

@@ -63,6 +63,9 @@ typedef enum {
 
 /* ========== Constants ========== */
 
+/** @brief Maximum number of camera devices */
+#define CCAP_MAX_DEVICES 32
+
 /** @brief Maximum length for device name including null terminator */
 #define CCAP_MAX_DEVICE_NAME_LENGTH 128
 
@@ -94,6 +97,12 @@ typedef struct {
     uint32_t height;
 } CcapResolution;
 
+/** @brief Device names list structure */
+typedef struct {
+    char deviceNames[CCAP_MAX_DEVICES][CCAP_MAX_DEVICE_NAME_LENGTH]; /**< Array of device names */
+    size_t deviceCount;                                              /**< Number of devices found */
+} CcapDeviceNamesList;
+
 /** @brief Device information structure */
 typedef struct {
     char deviceName[CCAP_MAX_DEVICE_NAME_LENGTH];      /**< Device name */
@@ -139,18 +148,28 @@ void ccap_provider_destroy(CcapProvider* provider);
 /* ========== Device Discovery ========== */
 
 /**
- * @brief Find all available camera device names
+ * @brief Find all available camera device names (new optimized API)
+ * @param provider Pointer to CcapProvider instance
+ * @param deviceList Output parameter for device names list
+ * @return true on success, false on failure
+ */
+bool ccap_provider_find_device_names_list(CcapProvider* provider, CcapDeviceNamesList* deviceList);
+
+/**
+ * @brief Find all available camera device names (legacy API - deprecated)
  * @param provider Pointer to CcapProvider instance
  * @param deviceNames Output array of device name strings (caller must free each string and the array)
  * @param count Output parameter for number of devices found
  * @return true on success, false on failure
+ * @deprecated Use ccap_provider_find_device_names_list instead
  */
 bool ccap_provider_find_device_names(CcapProvider* provider, char*** deviceNames, size_t* count);
 
 /**
- * @brief Free device names array returned by ccap_provider_find_device_names
+ * @brief Free device names array returned by ccap_provider_find_device_names (legacy API - deprecated)
  * @param deviceNames Array of device name strings
  * @param count Number of device names
+ * @deprecated Use ccap_provider_find_device_names_list instead
  */
 void ccap_provider_free_device_names(char** deviceNames, size_t count);
 

src/ccap_c.cpp

@@ -105,6 +105,32 @@ void ccap_provider_destroy(CcapProvider* provider) {
 
 /* ========== Device Discovery ========== */
 
+bool ccap_provider_find_device_names_list(CcapProvider* provider, CcapDeviceNamesList* deviceList) {
+    if (!provider || !deviceList) return false;
+
+    auto* cppProvider = reinterpret_cast<ccap::Provider*>(provider);
+    auto devices = cppProvider->findDeviceNames();
+
+    // Initialize structure
+    memset(deviceList, 0, sizeof(CcapDeviceNamesList));
+    
+    deviceList->deviceCount = devices.size();
+    if (deviceList->deviceCount > CCAP_MAX_DEVICES) {
+        deviceList->deviceCount = CCAP_MAX_DEVICES;
+    }
+
+    for (size_t i = 0; i < deviceList->deviceCount; ++i) {
+        size_t nameLen = devices[i].size();
+        if (nameLen >= CCAP_MAX_DEVICE_NAME_LENGTH) {
+            nameLen = CCAP_MAX_DEVICE_NAME_LENGTH - 1;
+        }
+        strncpy(deviceList->deviceNames[i], devices[i].c_str(), nameLen);
+        deviceList->deviceNames[i][nameLen] = '\0';
+    }
+
+    return true;
+}
+
 bool ccap_provider_find_device_names(CcapProvider* provider, char*** deviceNames, size_t* count) {
     if (!provider || !deviceNames || !count) return false;