Merge lp:~3v1n0/libunity/fix-ftb-wily into lp:libunity

Proposed by Marco Trevisan (Treviño)
Status: Merged
Approved by: Andrea Azzarone
Approved revision: 316
Merged at revision: 315
Proposed branch: lp:~3v1n0/libunity/fix-ftb-wily
Merge into: lp:libunity
Diff against target: 181 lines (+33/-34)
6 files modified
protocol/protocol-scope-discovery.vala (+4/-3)
src/unity-launcher.vala (+4/-8)
test/vala/test-launcher-integration.vala (+1/-2)
tools/music-track-model-renderer.vala (+5/-7)
tools/preview-renderer.vala (+11/-6)
tools/unity-tool-ui.vala (+8/-8)
To merge this branch: bzr merge lp:~3v1n0/libunity/fix-ftb-wily
Reviewer Review Type Date Requested Status
Andrea Azzarone (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+270824@code.launchpad.net

Commit message

Tools: use proper type prefix to fix FTB

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Andrea Azzarone (azzar1) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'protocol/protocol-scope-discovery.vala'
--- protocol/protocol-scope-discovery.vala 2013-10-01 13:32:26 +0000
+++ protocol/protocol-scope-discovery.vala 2015-09-11 13:57:34 +0000
@@ -269,10 +269,11 @@
269269
270 internal static void update_hidden_scope_ids ()270 internal static void update_hidden_scope_ids ()
271 {271 {
272 unowned string[] schemas = Settings.list_schemas ();272 var schema_src = SettingsSchemaSource.get_default().lookup (SCOPE_SCHEMA, false);
273 if (SCOPE_SCHEMA in schemas)273
274 if (schema_src != null)
274 {275 {
275 var settings = new Settings (SCOPE_SCHEMA);276 var settings = new Settings.full (schema_src, null, null);
276 hidden_scope_ids = settings.get_strv ("hidden-scopes");277 hidden_scope_ids = settings.get_strv ("hidden-scopes");
277 }278 }
278 else279 else
279280
=== modified file 'src/unity-launcher.vala'
--- src/unity-launcher.vala 2012-11-05 21:09:31 +0000
+++ src/unity-launcher.vala 2015-09-11 13:57:34 +0000
@@ -361,9 +361,11 @@
361 private LauncherFavorites ()361 private LauncherFavorites ()
362 {362 {
363 fav_cache = new HashTable<string,AppInfo?> (str_hash, str_equal);363 fav_cache = new HashTable<string,AppInfo?> (str_hash, str_equal);
364 if (schema_exists (LAUNCHER_SCHEMA_NAME))364 var schema_src = SettingsSchemaSource.get_default().lookup (LAUNCHER_SCHEMA_NAME, false);
365
366 if (schema_src != null)
365 {367 {
366 settings = new Settings (LAUNCHER_SCHEMA_NAME);368 var settings = new Settings.full (schema_src, null, null);
367 reset_fav_cache ();369 reset_fav_cache ();
368370
369 settings.changed["favorites"].connect (reset_fav_cache);371 settings.changed["favorites"].connect (reset_fav_cache);
@@ -374,12 +376,6 @@
374 }376 }
375 }377 }
376378
377 private static bool schema_exists (string schema)
378 {
379 unowned string[] installed_schemas = Settings.list_schemas ();
380 return schema in installed_schemas;
381 }
382
383 private void reset_fav_cache ()379 private void reset_fav_cache ()
384 {380 {
385 fav_cache.remove_all ();381 fav_cache.remove_all ();
386382
=== modified file 'test/vala/test-launcher-integration.vala'
--- test/vala/test-launcher-integration.vala 2012-11-05 21:09:31 +0000
+++ test/vala/test-launcher-integration.vala 2015-09-11 13:57:34 +0000
@@ -29,8 +29,7 @@
2929
30 public static bool schema_exists (string schema)30 public static bool schema_exists (string schema)
31 {31 {
32 unowned string[] installed_schemas = Settings.list_schemas ();32 return SettingsSchemaSource.get_default().lookup (schema, false) != null;
33 return schema in installed_schemas;
34 }33 }
3534
36 public static int main (string[] args)35 public static int main (string[] args)
3736
=== modified file 'tools/music-track-model-renderer.vala'
--- tools/music-track-model-renderer.vala 2013-07-22 12:04:23 +0000
+++ tools/music-track-model-renderer.vala 2015-09-11 13:57:34 +0000
@@ -17,8 +17,6 @@
17 *17 *
18 */18 */
1919
20using Gtk;
21
22namespace Unity.Tester {20namespace Unity.Tester {
2321
24public class MusicTrackModelRenderer: Object22public class MusicTrackModelRenderer: Object
@@ -27,13 +25,13 @@
27 private Dee.ModelTag<int> track_model_tag;25 private Dee.ModelTag<int> track_model_tag;
28 private int row_counter = 0;26 private int row_counter = 0;
29 private ulong model_sync_sig_id = 0;27 private ulong model_sync_sig_id = 0;
30 public ListStore track_view_model { get; construct; }28 public Gtk.ListStore track_view_model { get; construct; }
3129
32 public signal void track_list_synchronized();30 public signal void track_list_synchronized();
3331
34 construct32 construct
35 {33 {
36 track_view_model = new ListStore(6, typeof(string), typeof(int), typeof(string), typeof(uint), typeof(uint), typeof(double));34 track_view_model = new Gtk.ListStore(6, typeof(string), typeof(int), typeof(string), typeof(uint), typeof(uint), typeof(double));
37 }35 }
3836
39 public MusicTrackModelRenderer(Dee.Model track_model)37 public MusicTrackModelRenderer(Dee.Model track_model)
@@ -67,8 +65,8 @@
67 var row = model.get_row(iter);65 var row = model.get_row(iter);
6866
69 track_model_tag.set(track_model, iter, row_counter++);67 track_model_tag.set(track_model, iter, row_counter++);
70 68
71 TreeIter tm_iter;69 Gtk.TreeIter tm_iter;
72 track_view_model.append(out tm_iter);70 track_view_model.append(out tm_iter);
73 track_view_model.set(tm_iter, 0, row[0].get_string(), 1, row[1].get_int32(), 2, row[2].get_string(), 3, row[3].get_uint32(), 4, row[4].get_uint32(), 5, row[5].get_double(), -1);71 track_view_model.set(tm_iter, 0, row[0].get_string(), 1, row[1].get_int32(), 2, row[2].get_string(), 3, row[3].get_uint32(), 4, row[4].get_uint32(), 5, row[5].get_double(), -1);
74 }72 }
@@ -76,7 +74,7 @@
76 private void track_changed_cb(Dee.Model model, Dee.ModelIter iter)74 private void track_changed_cb(Dee.Model model, Dee.ModelIter iter)
77 {75 {
78 int index = track_model_tag.get(track_model, iter);76 int index = track_model_tag.get(track_model, iter);
79 TreeIter tm_iter;77 Gtk.TreeIter tm_iter;
80 if (track_view_model.get_iter_first(out tm_iter)) {78 if (track_view_model.get_iter_first(out tm_iter)) {
81 while (index > 0)79 while (index > 0)
82 {80 {
8381
=== modified file 'tools/preview-renderer.vala'
--- tools/preview-renderer.vala 2013-07-22 12:04:23 +0000
+++ tools/preview-renderer.vala 2015-09-11 13:57:34 +0000
@@ -148,12 +148,17 @@
148 icon_label.set_markup(name);148 icon_label.set_markup(name);
149 grid.attach(icon_label, 0, row, 1, 1);149 grid.attach(icon_label, 0, row, 1, 1);
150 var themed_icon = Gtk.IconTheme.get_default().lookup_by_gicon(icon, size, 0);150 var themed_icon = Gtk.IconTheme.get_default().lookup_by_gicon(icon, size, 0);
151 var pixbuf = themed_icon.load_icon();151 try {
152 Gtk.Image image = new Gtk.Image.from_pixbuf(pixbuf);152 var pixbuf = themed_icon.load_icon();
153 if (tooltip != null) {153 Gtk.Image image = new Gtk.Image.from_pixbuf(pixbuf);
154 image.set_tooltip_text(tooltip);154 if (tooltip != null) {
155 }155 image.set_tooltip_text(tooltip);
156 grid.attach(image, 1, row, 1, 1);156 }
157 grid.attach(image, 1, row, 1, 1);
158 }
159 catch (GLib.Error e) {
160 warning(@"Got error while loading pixmap: $(e.message)");
161 }
157162
158 ++row;163 ++row;
159 }164 }
160165
=== modified file 'tools/unity-tool-ui.vala'
--- tools/unity-tool-ui.vala 2013-09-24 08:57:39 +0000
+++ tools/unity-tool-ui.vala 2015-09-11 13:57:34 +0000
@@ -40,9 +40,9 @@
40 window.destroy.connect(Gtk.main_quit);40 window.destroy.connect(Gtk.main_quit);
41 window.show_all();41 window.show_all();
4242
43 uimodel = builder.get_object("results_model") as ListStore;43 uimodel = builder.get_object("results_model") as Gtk.ListStore;
44 ui_filter_model = builder.get_object("filters_model") as ListStore;44 ui_filter_model = builder.get_object("filters_model") as Gtk.ListStore;
45 ui_cat_model = builder.get_object("categories_model") as ListStore;45 ui_cat_model = builder.get_object("categories_model") as Gtk.ListStore;
4646
47 notebook = builder.get_object("notebook") as Notebook;47 notebook = builder.get_object("notebook") as Notebook;
48 search_entry = builder.get_object("search_entry") as Entry;48 search_entry = builder.get_object("search_entry") as Entry;
@@ -155,7 +155,7 @@
155 builder.add_from_resource("/com/canonical/Unity/unity-tool/dbus-scope-connect.ui");155 builder.add_from_resource("/com/canonical/Unity/unity-tool/dbus-scope-connect.ui");
156 builder.connect_signals(this);156 builder.connect_signals(this);
157 scope_discovery_spinner = builder.get_object("scope_discovery_spinner") as Spinner;157 scope_discovery_spinner = builder.get_object("scope_discovery_spinner") as Spinner;
158 scope_list_model = builder.get_object("scope_list_model") as ListStore;158 scope_list_model = builder.get_object("scope_list_model") as Gtk.ListStore;
159 scope_list_combobox = builder.get_object("scope_list_combobox") as ComboBox;159 scope_list_combobox = builder.get_object("scope_list_combobox") as ComboBox;
160 dbus_name_entry = builder.get_object("dbus_name_entry") as Entry;160 dbus_name_entry = builder.get_object("dbus_name_entry") as Entry;
161 dbus_path_entry = builder.get_object("dbus_path_entry") as Entry;161 dbus_path_entry = builder.get_object("dbus_path_entry") as Entry;
@@ -843,10 +843,10 @@
843 private Gtk.Entry dbus_name_entry = null;843 private Gtk.Entry dbus_name_entry = null;
844 private Gtk.Entry dbus_path_entry = null;844 private Gtk.Entry dbus_path_entry = null;
845 private Gtk.ComboBox scope_list_combobox = null;845 private Gtk.ComboBox scope_list_combobox = null;
846 private ListStore uimodel = null;846 private Gtk.ListStore uimodel = null;
847 private ListStore ui_filter_model = null;847 private Gtk.ListStore ui_filter_model = null;
848 private ListStore scope_list_model = null;848 private Gtk.ListStore scope_list_model = null;
849 private ListStore ui_cat_model = null;849 private Gtk.ListStore ui_cat_model = null;
850 private ulong model_sync_sig_id;850 private ulong model_sync_sig_id;
851 private Dee.SharedModel? dee_results_model = null;851 private Dee.SharedModel? dee_results_model = null;
852 private string current_channel_id = "";852 private string current_channel_id = "";

Subscribers

People subscribed via source and target branches