Complete API optimization - remove legacy functions completely

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

Author: Copilot

include/ccap_c.h

@@ -148,31 +148,13 @@ void ccap_provider_destroy(CcapProvider* provider);
 /* ========== Device Discovery ========== */
 
 /**
- * @brief Find all available camera device names (new optimized API)
+ * @brief Find all available camera device names
  * @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 (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);
-
 /* ========== Device Management ========== */
 
 /**

src/ccap_c.cpp

@@ -47,16 +47,6 @@ CcapFrameOrientation convert_frame_orientation_to_c(ccap::FrameOrientation orien
     return static_cast<CcapFrameOrientation>(static_cast<uint32_t>(orientation));
 }
 
-// Helper to allocate and copy C string
-char* allocate_c_string(const std::string& str) {
-    if (str.empty()) return nullptr;
-    char* result = static_cast<char*>(malloc(str.length() + 1));
-    if (result) {
-        strcpy(result, str.c_str());
-    }
-    return result;
-}
-
 // Wrapper struct for callback management
 struct CallbackWrapper {
     CcapNewFrameCallback callback;
@@ -131,46 +121,6 @@ bool ccap_provider_find_device_names_list(CcapProvider* provider, CcapDeviceName
     return true;
 }
 
-bool ccap_provider_find_device_names(CcapProvider* provider, char*** deviceNames, size_t* count) {
-    if (!provider || !deviceNames || !count) return false;
-
-    auto* cppProvider = reinterpret_cast<ccap::Provider*>(provider);
-    auto devices = cppProvider->findDeviceNames();
-
-    *count = devices.size();
-    if (*count == 0) {
-        *deviceNames = nullptr;
-        return true;
-    }
-
-    char** names = static_cast<char**>(malloc(*count * sizeof(char*)));
-    if (!names) return false;
-
-    for (size_t i = 0; i < *count; ++i) {
-        names[i] = allocate_c_string(devices[i]);
-        if (!names[i]) {
-            // Cleanup on failure
-            for (size_t j = 0; j < i; ++j) {
-                free(names[j]);
-            }
-            free(names);
-            return false;
-        }
-    }
-
-    *deviceNames = names;
-    return true;
-}
-
-void ccap_provider_free_device_names(char** deviceNames, size_t count) {
-    if (deviceNames) {
-        for (size_t i = 0; i < count; ++i) {
-            free(deviceNames[i]);
-        }
-        free(deviceNames);
-    }
-}
-
 /* ========== Device Management ========== */
 
 bool ccap_provider_open(CcapProvider* provider, const char* deviceName, bool autoStart) {