unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#36167: [PATCH] Replace manually crafted hex regexes with [[:xdigit:]]
@ 2019-06-11 11:59 Konstantin Kharlamov
  2019-06-11 12:03 ` bug#36167: Acknowledgement ([PATCH] Replace manually crafted hex regexes with [[:xdigit:]]) Konstantin Kharlamov
                   ` (6 more replies)
  0 siblings, 7 replies; 42+ messages in thread
From: Konstantin Kharlamov @ 2019-06-11 11:59 UTC (permalink / raw)
  To: 36167

Hopefully, this is more readable compared to older "[0-9a-fA-F]", 
because the intention is immediately obvious.

Fun fact: initially I started it because on my older Emacs this 
resulted in better performance; the difference was about 10x times. But 
I can't reproduce this on the current master anymore, i.e. the 
performance matches. Oh, well. I'm thus not sending the patch replacing 
[0-9] with [[:digit:]] because in terms of readability the intention is 
clear there, however replacing the hex regexes should still make code 
more readable.

P.S.: I'm not sure if the commit message looks okay, comments are 
welcome.







^ permalink raw reply	[flat|nested] 42+ messages in thread

end of thread, other threads:[~2019-07-08 12:11 UTC | newest]

Thread overview: 42+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-06-11 11:59 bug#36167: [PATCH] Replace manually crafted hex regexes with [[:xdigit:]] Konstantin Kharlamov
2019-06-11 12:03 ` bug#36167: Acknowledgement ([PATCH] Replace manually crafted hex regexes with [[:xdigit:]]) Konstantin Kharlamov
2019-06-11 12:29 ` bug#36167: [PATCH] Replace [0-9a-fA-F] with [[:xdigit:]] Konstantin Kharlamov
2019-06-11 13:03   ` Robert Pluim
2019-06-11 13:17     ` Konstantin Kharlamov
2019-06-11 13:30       ` Robert Pluim
2019-06-12  0:29         ` Konstantin Kharlamov
2019-06-12  2:42           ` Eli Zaretskii
2019-06-12  8:21           ` Robert Pluim
2019-06-12 11:19             ` Konstantin Kharlamov
2019-06-12 14:55               ` Eli Zaretskii
2019-06-12 16:39                 ` Konstantin Kharlamov
2019-06-12 16:51                   ` Eli Zaretskii
2019-06-12 17:09                     ` Konstantin Kharlamov
2019-06-12 21:33                       ` Lars Ingebrigtsen
2019-06-12 21:39                         ` Lars Ingebrigtsen
2019-06-12 22:17                         ` Andreas Schwab
2019-06-12 22:22                           ` Lars Ingebrigtsen
2019-06-12 22:48                             ` Konstantin Kharlamov
2019-06-11 13:32     ` npostavs
2019-06-11 14:36       ` Eli Zaretskii
2019-06-11 14:26 ` bug#36167: [PATCH] Replace manually crafted hex regexes " Paul Eggert
2019-06-11 14:37   ` Konstantin Kharlamov
2019-06-11 15:37     ` Paul Eggert
2019-06-11 16:31       ` Mattias Engdegård
2019-06-11 16:50         ` Drew Adams
2019-06-11 17:04           ` Eli Zaretskii
2019-06-11 17:18             ` Paul Eggert
2019-06-11 17:32               ` Eli Zaretskii
2019-06-11 17:19             ` Drew Adams
2019-06-11 17:34               ` Eli Zaretskii
2019-06-11 17:50             ` npostavs
2019-06-11 17:53               ` Eli Zaretskii
2019-06-11 18:14                 ` Eli Zaretskii
2019-06-11 19:56 ` Andy Moreton
2019-06-12  2:30   ` Eli Zaretskii
2019-06-12 11:44 ` Andy Moreton
2019-06-12 16:07   ` Eli Zaretskii
2019-06-13  0:18 ` bug#36167: [PATCH v2] Replace manually crafted hex regexes wth [[:xdigit:]] Konstantin Kharlamov
2019-07-06  8:04   ` Eli Zaretskii
2019-07-08  9:34 ` bug#36167: [PATCH v2] Replace manually crafted hex regexes with [[:xdigit:]] Konstantin Kharlamov
2019-07-08 12:11   ` Eli Zaretskii

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).