unity-lens-applications

Merge lp:~didrocks/unity-lens-applications/fix-spaces-in-command-and-blame-mikkel into lp:unity-lens-applications

  1. fix-spaces-in-command-and-blame-mikkel
  2. Merge into trunk
Proposed by Didier Roche-Tolomelli
Status: Merged
Merged at revision: 209
Proposed branch: lp:~didrocks/unity-lens-applications/fix-spaces-in-command-and-blame-mikkel
Merge into: lp:unity-lens-applications
Diff against target: 12 lines (+1/-1)
1 file modified
src/daemon.vala (+1/-1)
To merge this branch: bzr merge lp:~didrocks/unity-lens-applications/fix-spaces-in-command-and-blame-mikkel
Reviewer Review Type Date Requested Status
Mikkel Kamstrup Erlandsen (community) Approve
Review via email: mp+59920@code.launchpad.net

Description of the change

ensure we are matching files as files (no space in the command). This enables to
launch gvim rrr:rrr.txt which is matched (LP: #776731)

To post a comment you must log in.
Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

 review approve

I saw that! ;-)

review: Approve
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Come on! I can't blame you silently? :-)
Thanks merging ;)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/daemon.vala'
2--- src/daemon.vala 2011-04-07 09:37:36 +0000
3+++ src/daemon.vala 2011-05-04 13:21:48 +0000
4@@ -183,7 +183,7 @@
5 runner = new Unity.ApplicationsPlace.Runner (this);
6
7 try {
8- uri_regex = new Regex ("[a-z]+:.+");
9+ uri_regex = new Regex ("^[a-z]+:.+$");
10 } catch (RegexError e) {
11 uri_regex = null;
12 critical ("Failed to compile URI regex. URL launching will be disabled");

Subscribers

People subscribed via source and target branches