lp:~ted/libindicator/lp719457

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:libindicator/0.5

Charles Kerr (community): Approve on 2012-03-05

Diff: 14 lines (+3/-1)

1 file modified

libindicator/indicator-service.c (+3/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #719457: valgrind invalid reads error on unloading

Low

Fix Released

Link a bug report

Related blueprints

456. By Ted Gould on 2012-03-02

Attaching bug

455. By Ted Gould on 2012-03-02

Copying the name before using it to remove as it seems that the unwatch can cause the name to be free'd in some cases. Odd, but valgrind can find it

454. By Ted Gould on 2012-03-02

Support new FD.o Desktop Actions

453. By Charles Kerr on 2012-02-29

lp:~charlesk/libindicator/fix-939061

452. By Ted Gould on 2012-02-23

0.4.92

451. By Ted Gould on 2012-02-23

Remove unenforcable consts

450. By Ted Gould on 2012-02-15

Adjusting timeout so it'll build on the buildds too

449. By Ted Gould on 2012-02-14

Explicitly listing gmodule as a library we're using

448. By Ted Gould on 2012-02-14

0.4.91

447. By Lars Karlitski on 2012-02-13

indicator_object_set_visible: only emit entry-{added,removed} when visibility actually changed

Emitting those signals is semantically incorrect. It also confused
indicator-loader and the panel, because they are not checking whether an entry
is already present in their entry-added handlers. This led to gtk warnings
("widget already has a parent") and strange effects, such as the same menu
being added multiple times into the panel.

This patch also makes indicator_object_dispose use indicator_object_set_visible
(it was already doing something similar).