Merge lp:~charlesk/indicator-datetime/free-builtin-timezones into lp:indicator-datetime/0.4

Proposed by Charles Kerr
Status: Merged
Approved by: Lars Karlitski
Approved revision: 165
Merged at revision: 167
Proposed branch: lp:~charlesk/indicator-datetime/free-builtin-timezones
Merge into: lp:indicator-datetime/0.4
Diff against target: 11 lines (+2/-0)
1 file modified
src/datetime-service.c (+2/-0)
To merge this branch: bzr merge lp:~charlesk/indicator-datetime/free-builtin-timezones
Reviewer Review Type Date Requested Status
Lars Karlitski (community) Approve
Review via email: mp+99583@code.launchpad.net

Description of the change

Minor change to free ical's internal list of timezones upon app shutdown

note to packagers: this change on its own doesn't merit a new release this week; it just makes the valgrind log easier to read.

To post a comment you must log in.
Revision history for this message
Lars Karlitski (larsu) wrote :

This seems like a good idea ;)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/datetime-service.c'
2--- src/datetime-service.c 2012-03-20 20:32:19 +0000
3+++ src/datetime-service.c 2012-03-27 18:36:19 +0000
4@@ -1479,6 +1479,8 @@
5 g_object_unref(G_OBJECT(server));
6 g_object_unref(G_OBJECT(root));
7
8+ icaltimezone_free_builtin_timezones();
9+
10 geo_address_clean();
11 geo_client_clean();
12

Subscribers

People subscribed via source and target branches