GNU bug report logs - #73307
Fix ctype(3) usage

Previous Next

Package: emacs;

Reported by: Thomas Klausner <wiz <at> gatalith.at>

Date: Mon, 16 Sep 2024 21:26:01 UTC

Severity: normal

Tags: patch

Done: Paul Eggert <eggert <at> cs.ucla.edu>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: luangruo <at> yahoo.com, 73307 <at> debbugs.gnu.org, wiz <at> gatalith.at, monnier <at> iro.umontreal.ca
Subject: bug#73307: Fix ctype(3) usage
Date: Wed, 18 Sep 2024 19:11:02 +0300
> Date: Wed, 18 Sep 2024 08:35:18 -0700
> Cc: 73307 <at> debbugs.gnu.org, wiz <at> gatalith.at, luangruo <at> yahoo.com
> From: Paul Eggert <eggert <at> cs.ucla.edu>
> 
> > You are testing characters for whitespace syntax, which is AFAIU
> > subject to buffer-local syntax tables.
> 
> Thanks, I didn't think of that. How about the attached patch? It is an 
> alternative to just using c_isspace.

That's much better, thanks.  But I'm not sure we really want to allow
any Zs character there.  For example, what about U+3000 IDEOGRAPHIC
SPACE?  But if this is okay with us, please install.




This bug report was last modified 296 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.