Feature #11949: Allow @/$ prefix in Regexp's named captures

• Status: Closed
• Assignee: -
• Target version: -

[repeatedly (Masahiro Nakagawa)]

fluentd hits the problem that Regexp's named captures doesn't allow @ prefix.

https://github.com/treasure-data/omnibus-td-agent/issues/60

I assume this limitation is for auto variable assign feature.

http://ruby-doc.org/core-2.3.0/Regexp.html#class-Regexp-label-Capturing

In fluentd, there is no auto assign use case so we want to use @/$ prefix.
@ is used for Elastic related products and $ is used for MongoDB.
Currently, we need to add extra routine to convert foo to @foo / $foo after parsed the data.
It makes performance slower and causes complicated configuration.

I'm not sure the best approach but we want command line option or something to allow @/$ prefix in named captures.

[matz (Yukihiro Matsumoto)]

It should be fixed,I think,without assignment behavior.

Matz.

[naruse (Yui NARUSE)]

• Status changed from Open to Closed

Applied in changeset r53610.

---

• regparse.c (fetch_name_with_level): allow non word characters
  at the first character. [Feature #11949]

• regparse.c (fetch_name): ditto.

[naruse (Yui NARUSE)]

I committed to allow all characters (though previous phase rejects characters conflicting look behind (<! and <=)).

Anyway I notice original Oniguruma looks intended to allow only WORD.
I sent PR to Onigmo.
https://github.com/k-takata/Onigmo/pull/62