libappindicator

Merge lp://qastaging/~mitya57/libappindicator/test-simple-app into lp://qastaging/libappindicator/15.10

  1. test-simple-app
  2. Merge into trunk.15.10
Proposed by Dmitry Shachnev
Status: Superseded
Proposed branch: lp://qastaging/~mitya57/libappindicator/test-simple-app
Merge into: lp://qastaging/libappindicator/15.10
Diff against target: 48 lines (+12/-12)
1 file modified
tests/test-simple-app.c (+12/-12)
To merge this branch: bzr merge lp://qastaging/~mitya57/libappindicator/test-simple-app
Reviewer Review Type Date Requested Status
Ted Gould (community) Approve
Alberts Muktupāvels Approve
Review via email: mp+263186@code.qastaging.launchpad.net

This proposal has been superseded by a proposal from 2016-09-05.

Description of the change

The previous version of the simple test application was crashing with SIGSEGV in g_object_new call. This one seems to actually work.

To post a comment you must log in.
Revision history for this message
Alberts Muktupāvels (muktupavels) wrote :

I probably would use gtk_menu_shell_append not gtk_menu_attach, but that should not be problem.

Code looks good, but I did not test it.

review: Approve
lp://qastaging/~mitya57/libappindicator/test-simple-app updated
274. By Dmitry Shachnev

Make test-simple-app really working

Revision history for this message
Dmitry Shachnev (mitya57) wrote :

gtk_menu_shell_append is of course better, updated

Revision history for this message
Ted Gould (ted) :
review: Approve

Unmerged revisions

274. By Dmitry Shachnev

Make test-simple-app really working

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches