[haiku-commits] Re: haiku: hrev50089 - src/apps/people

  • From: François Revol <revol@xxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 16 Feb 2016 03:02:41 +0100

On 14/02/2016 14:28, pulkomandy@xxxxxxxxxxxxx wrote:

hrev50089 adds 1 changeset to branch 'master'
old head: a5a3b2d9a3d95cbae71eaf371708c73a1780ac0d
new head: 632e56d8e514ba6ac41f582ce580e51a3cd8922e
overview: 
http://cgit.haiku-os.org/haiku/log/?qt=range&q=632e56d8e514+%5Ea5a3b2d9a3d9

----------------------------------------------------------------------------

632e56d8e514: URL linkification in People
  
  url and email label are now marked as a link and open the address
  in the browser/mail-app on click
  
  Signed-off-by: Adrien Destugues <pulkomandy@xxxxxxxxxxxxx>
  
  Fixes #3825
  
  A few extra changes from the original patch:
  
  - Remove "smart" parsing (detection of "ftp", etc) as it could lead to
  ftp://ftp://url or other strangeness,
  - Add gopher protocol, because mmu_man may have an home page there.

Thanks :D

+{
+     if (url.IFindFirst("http://";) == 0 || url.IFindFirst("ftp://";) == 0
+             || url.IFindFirst("https://";) || url.IFindFirst("gopher://";) == 
0) {

Nice, although shouldn't it just check for [A-Za-z0-9+] before :// and
test for existing MIME with it instead of hardcoding the list?

François.

Other related posts: