Refactor menu item API to use nullable setters

Removed explicit remove methods for menu item accelerators and submenus in favor of passing NULL to setter functions. Updated documentation and usage to reflect this change, simplifying the API and improving consistency.

Author: lijy91

packages/cnativeapi/lib/src/bindings_generated.dart

@@ -836,7 +836,7 @@ class CNativeApiBindings {
 
   /// Set the keyboard accelerator for a menu item
   /// @param item The menu item
-  /// @param accelerator The keyboard accelerator to set
+  /// @param accelerator The keyboard accelerator to set, or NULL to remove the accelerator
   void native_menu_item_set_accelerator(
     native_menu_item_t item,
     ffi.Pointer<native_keyboard_accelerator_t> accelerator,
@@ -891,20 +891,6 @@ class CNativeApiBindings {
             )
           >();
 
-  /// Remove the keyboard accelerator from a menu item
-  /// @param item The menu item
-  void native_menu_item_remove_accelerator(native_menu_item_t item) {
-    return _native_menu_item_remove_accelerator(item);
-  }
-
-  late final _native_menu_item_remove_acceleratorPtr =
-      _lookup<ffi.NativeFunction<ffi.Void Function(native_menu_item_t)>>(
-        'native_menu_item_remove_accelerator',
-      );
-  late final _native_menu_item_remove_accelerator =
-      _native_menu_item_remove_acceleratorPtr
-          .asFunction<void Function(native_menu_item_t)>();
-
   /// Set the enabled state of a menu item
   /// @param item The menu item
   /// @param enabled true to enable, false to disable
@@ -1000,7 +986,7 @@ class CNativeApiBindings {
 
   /// Set the submenu for a menu item
   /// @param item The menu item
-  /// @param submenu The submenu to attach
+  /// @param submenu The submenu to attach, or NULL to remove the submenu
   void native_menu_item_set_submenu(
     native_menu_item_t item,
     native_menu_t submenu,
@@ -1029,20 +1015,6 @@ class CNativeApiBindings {
   late final _native_menu_item_get_submenu = _native_menu_item_get_submenuPtr
       .asFunction<native_menu_t Function(native_menu_item_t)>();
 
-  /// Remove the submenu from a menu item
-  /// @param item The menu item
-  void native_menu_item_remove_submenu(native_menu_item_t item) {
-    return _native_menu_item_remove_submenu(item);
-  }
-
-  late final _native_menu_item_remove_submenuPtr =
-      _lookup<ffi.NativeFunction<ffi.Void Function(native_menu_item_t)>>(
-        'native_menu_item_remove_submenu',
-      );
-  late final _native_menu_item_remove_submenu =
-      _native_menu_item_remove_submenuPtr
-          .asFunction<void Function(native_menu_item_t)>();
-
   /// Add event listener for a menu item
   /// @param item The menu item
   /// @param event_type The type of event to listen for

packages/cnativeapi/src/libnativeapi

@@ -88,18 +88,15 @@ class MenuItem
   static bool _callbacksInitialized = false;
 
   late final native_menu_item_t _nativeHandle;
-  
+
   // Store listener IDs for cleanup
   int? _clickedListenerId;
   int? _submenuOpenedListenerId;
   int? _submenuClosedListenerId;
 
   MenuItem(String label, [MenuItemType type = MenuItemType.normal]) {
     final labelPtr = label.toNativeUtf8().cast<Char>();
-    _nativeHandle = bindings.native_menu_item_create(
-      labelPtr,
-      type.toNative(),
-    );
+    _nativeHandle = bindings.native_menu_item_create(labelPtr, type.toNative());
     ffi.calloc.free(labelPtr);
 
     // Store instance in static map using handle address as key
@@ -292,7 +289,7 @@ class MenuItem
 
   set submenu(Menu? menu) {
     if (menu == null) {
-      bindings.native_menu_item_remove_submenu(_nativeHandle);
+      bindings.native_menu_item_set_submenu(_nativeHandle, nullptr);
     } else {
       bindings.native_menu_item_set_submenu(_nativeHandle, menu.nativeHandle);
     }
@@ -329,7 +326,7 @@ class Menu
   _closedCallback;
 
   static bool _callbacksInitialized = false;
-  
+
   // Store listener IDs for cleanup
   int? _openedListenerId;
   int? _closedListenerId;
@@ -375,17 +372,11 @@ class Menu
   void stopEventListening() {
     // Remove native listeners using stored IDs
     if (_openedListenerId != null) {
-      bindings.native_menu_remove_listener(
-        _nativeHandle,
-        _openedListenerId!,
-      );
+      bindings.native_menu_remove_listener(_nativeHandle, _openedListenerId!);
       _openedListenerId = null;
     }
     if (_closedListenerId != null) {
-      bindings.native_menu_remove_listener(
-        _nativeHandle,
-        _closedListenerId!,
-      );
+      bindings.native_menu_remove_listener(_nativeHandle, _closedListenerId!);
       _closedListenerId = null;
     }
   }