unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Kenichi Handa <handa@m17n.org>
Cc: rv@gnu.org, emacs-devel@gnu.org
Subject: Re: ffap not UTF-8 ready
Date: Tue, 03 Oct 2006 10:20:42 +0900	[thread overview]
Message-ID: <E1GUYxe-0001es-00@etlken> (raw)
In-Reply-To: <87y7ryjuml.fsf@zip.com.au> (message from Kevin Ryde on Tue, 03 Oct 2006 07:05:54 +1000)

In article <87y7ryjuml.fsf@zip.com.au>, Kevin Ryde <user42@zip.com.au> writes:

> Kenichi Handa <handa@m17n.org> writes:
> >
> > It seems that this is because the variable
> > ffap-string-at-point-mode-alist doesn't contain a multibyte
> > character in CHARS.

> Perhaps "(thing-at-point 'filename)", in thing-at-point-file-name-chars,
> has the same problem.

!! The variable thing-at-point-file-name-chars is defined as
    "-~/[:alnum:]_.${}#%,:".
I've forgotten about [:XXX:] notation.  I've just read
src/regex.c and found that [:alnum:] also works for
multibyte characters (it matches with a multibyte character
whose syntax is "word"), and [:multibyte:] is available too.
So, the current definition of thing-at-point-file-name-chars
works in most cases.  But, considering that a non-word
multibyte character can also be used in a file name,
perhaps, defining that as
   "-~/[:alnum:][:multibyte:]_.${}#%,:"
is better.

And, I think ffap.el should also use that kind of pattern
instead of something like this: "0-9A-Za-z".

---
Kenichi Handa
handa@m17n.org

  reply	other threads:[~2006-10-03  1:20 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <E1GSrVS-0000rW-Qr@jidanni.org>
2006-10-02  7:22 ` ffap not UTF-8 ready Kenichi Handa
2006-10-02 13:49   ` Stefan Monnier
2006-10-02 21:05   ` Kevin Ryde
2006-10-03  1:20     ` Kenichi Handa [this message]
     [not found]       ` <E1GUliX-0003XK-Lx@fencepost.gnu.org>
2006-10-03 23:26         ` Kenichi Handa
2006-10-04 16:22           ` Richard Stallman
2006-10-11 20:29           ` Richard Stallman
2006-10-12 15:48             ` Rajesh Vaidheeswarran

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=E1GUYxe-0001es-00@etlken \
    --to=handa@m17n.org \
    --cc=emacs-devel@gnu.org \
    --cc=rv@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).