Bug#415849: rxvt-unicode: url parsing of mark-urls appears to be suspect

2007-12-13 Thread Reuben Thomas

On Thu, 13 Dec 2007, Decklin Foster wrote:


Can you try this with 8.7-1? I was looking at the selection extension
and this seems partially fixed. In the first case,

http://foo.com/

is selected, and in the second (which is the same as before IIRC)

http://www.foo.com/.

I'm not sure what would be considered right there, but I'm inclined to
say that since . is a valid character in a URL it should be selected.


On logical grounds I'm inclined to agree. OTOH, it's fairly common to see 
cases where the . is intended to be a sentence period, not a part of the 
URL.


I agree that my original complaint seems fixed.

--
http://rrt.sc3d.org/ | compulsion, n.  the eloquence of power (Bierce)



--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#415849: rxvt-unicode: url parsing of mark-urls appears to be suspect

2007-12-12 Thread Decklin Foster
Reuben Thomas writes:

 The problem I noticed is that in the following text:
 
 (http://foo.com/).
 
 this URL is highlighted:
 
 http://foo.com/)
 
 which is unlikely to be right, yet omits a potentially valid
 character, .. (I'm not sure whether parentheses, on the other hand,
 are legal.)
 
 I'd suggest that trailing punctuation be ignored. This would help not
 only with this case, but a case like:
 
 See our site at http://www.foo.com/.;

Can you try this with 8.7-1? I was looking at the selection extension
and this seems partially fixed. In the first case,

http://foo.com/

is selected, and in the second (which is the same as before IIRC)

http://www.foo.com/.

I'm not sure what would be considered right there, but I'm inclined to
say that since . is a valid character in a URL it should be selected.

-- 
things change.
[EMAIL PROTECTED]



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#415849: rxvt-unicode: url parsing of mark-urls appears to be suspect

2007-03-22 Thread Reuben Thomas
Package: rxvt-unicode
Version: 7.9-2
Severity: minor

As ever, I suspect the problem is trying to balance matching things
that can possibly be valid URLs against things that aren't intended to
be.

The problem I noticed is that in the following text:

(http://foo.com/).

this URL is highlighted:

http://foo.com/)

which is unlikely to be right, yet omits a potentially valid
character, .. (I'm not sure whether parentheses, on the other hand,
are legal.)

I'd suggest that trailing punctuation be ignored. This would help not
only with this case, but a case like:

See our site at http://www.foo.com/.;

-- System Information:
Debian Release: 4.0
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/bash
Kernel: Linux 2.6.18-4-686
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)

Versions of packages rxvt-unicode depends on:
ii  base-passwd 3.5.11   Debian base system master password
ii  libc6   2.3.6.ds1-13 GNU C Library: Shared libraries
ii  libfontconfig1  2.4.2-1.2generic font configuration library
ii  libgcc1 1:4.1.1-21   GCC support library
ii  libperl5.8  5.8.8-7  Shared Perl library
ii  libx11-62:1.0.3-6X11 client-side library
ii  libxft2 2.1.8.2-8FreeType-based font drawing librar
ii  libxpm4 1:3.5.5-2X11 pixmap library
ii  libxrender1 1:0.9.1-3X Rendering Extension client libra
ii  ncurses-base5.5-5Descriptions of common terminal ty

rxvt-unicode recommends no packages.

-- no debconf information


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]