Merge lp:~charlesk/unity-gtk-module/lp-664647 into lp:unity-gtk-module/14.04

Proposed by Charles Kerr
Status: Merged
Approved by: William Hua
Approved revision: 306
Merged at revision: 306
Proposed branch: lp:~charlesk/unity-gtk-module/lp-664647
Merge into: lp:unity-gtk-module/14.04
Diff against target: 19 lines (+3/-0)
1 file modified
src/main.c (+3/-0)
To merge this branch: bzr merge lp:~charlesk/unity-gtk-module/lp-664647
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
William Hua (community) Approve
Review via email: mp+193461@code.launchpad.net

Description of the change

Add acroread to the blacklist.

To post a comment you must log in.
Revision history for this message
William Hua (attente) wrote :

Looks good!

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/main.c'
2--- src/main.c 2013-03-14 17:09:54 +0000
3+++ src/main.c 2013-10-31 17:36:57 +0000
4@@ -90,6 +90,8 @@
5 gint *natural_height);
6 #endif
7
8+/* Crude blacklist to avoid patching innocent apps.
9+ Use xprop | grep CLASS to find the name to use */
10 static gboolean
11 is_blacklisted (const gchar *name)
12 {
13@@ -100,6 +102,7 @@
14 g_strcmp0 (name, "emacs24") == 0 ||
15 g_strcmp0 (name, "emacs24-lucid") == 0 ||
16 g_strcmp0 (name, "glade") == 0 ||
17+ g_strcmp0 (name, "acroread") == 0 ||
18 g_strcmp0 (name, "gwyddion") == 0;
19 }
20

Subscribers

People subscribed via source and target branches