Skip to content

[Gtk4] Set proper MenuItem.section on destroy#3297

Merged
akurtakov merged 1 commit into
eclipse-platform:masterfrom
akurtakov:master
May 12, 2026
Merged

[Gtk4] Set proper MenuItem.section on destroy#3297
akurtakov merged 1 commit into
eclipse-platform:masterfrom
akurtakov:master

Conversation

@akurtakov
Copy link
Copy Markdown
Member

Fixes many problems like the following on editor switching:

(Eclipse:326795): GLib-GIO-CRITICAL **: 13:57:51.527:
g_menu_insert_item: assertion 'G_IS_MENU_ITEM (item)' failed

(Eclipse:326795): GLib-GIO-CRITICAL **: 13:57:51.527: g_menu_remove:
assertion '0 <= position && (guint) position < menu->items->len' failed

Fixes many problems like the following on editor switching:
```
(Eclipse:326795): GLib-GIO-CRITICAL **: 13:57:51.527:
g_menu_insert_item: assertion 'G_IS_MENU_ITEM (item)' failed

(Eclipse:326795): GLib-GIO-CRITICAL **: 13:57:51.527: g_menu_remove:
assertion '0 <= position && (guint) position < menu->items->len' failed
```
@akurtakov akurtakov self-assigned this May 12, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Test Results (linux)

   94 files  ±0     94 suites  ±0   14m 25s ⏱️ +7s
4 597 tests ±0  4 373 ✅ ±0  224 💤 ±0  0 ❌ ±0 
3 383 runs  ±0  3 308 ✅ ±0   75 💤 ±0  0 ❌ ±0 

Results for commit 9e95680. ± Comparison against base commit 4277d11.

@akurtakov
Copy link
Copy Markdown
Member Author

Merging this one as it's strictly Gtk 4 specific and reduces glib/gtk critical issues output a lot thus allowing to further investigate remaining problems.

@akurtakov akurtakov merged commit c5244c1 into eclipse-platform:master May 12, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant