Bug#415849: rxvt-unicode: url parsing of mark-urls appears to be suspect
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
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
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]