[IDEA] Regex: Implement named captures

[ma8ma]

edit(2022-01-29): 上流にマージされました (JDimproved#904)

---

このPRはアイデア検証用でありマージしません。

正規表現の名前付きキャプチャ機能をラッパークラスに実装します。
Glib Regexのみ有効でそれ以外の正規表現ライブラリでは名前を指定してもマッチしません。

合わせて スレタイ検索時にアドレスとスレタイを取得する正規表現 (about:config) に名前付きキャプチャ対応を追加します。
また、互換性のためグループ番号は現状を維持します。

スレタイ検索の設定

スレタイ検索時にアドレスとスレタイを取得する正規表現 (about:config) はグループ番号のかわりに名前付きキャプチャを利用することができる。( v0.6.0+から追加、Glib Regexのみ )

正規表現パターン中に名前が有れば名前付きキャプチャ、無ければグループ番号で対象を取得する。

取得対象	グループ番号	キャプチャの名前	備考
スレURL	1	url	必須
スレタイトル	2	subject	必須
レス数	3	number	オプション

グループ番号は順序固定のため取得対象が番号順に並んでいないときは名前付きキャプチャを使うと分かりやすい。
(下の例はスレタイ、レス数、URLの順に並んでいるデータにマッチする)

設定例	正規表現パターン
グループ番号	^(?=[^\t\n]+\t[^\t\n]+\t([^\t\n]+))([^\t\n]+)\t([^\t\n]+)
名前付きキャプチャ	^(?<subject>[^\t\n]+)\t(?<number>[^\t\n]+)\t(?<url>[^\t\n]+)

正規表現の構文はGlib公式など外部サイトを参照。

---

パッチの適応方法

git clone コマンドを使う方法 (一時的なcloneで使い捨てる)

ブランチを削除したため利用できません

git clone -b idea-regex-implement-named-capture --depth 1 https://github.com/ma8ma/JDim.git temp
cd temp

meson debugdir
meson compile -C debugdir

# 実行
./debugdir/src/jdim

# パッチの削除
cd ../
rm -rf temp

curl と patch コマンドを使う場合

※ git pullなどでmasterブランチを更新してから行うことを推奨します。

git chechout master
git pull
curl -L https://github.com/ma8ma/JDim/pull/59.patch | patch -p1
meson debugdir
meson compile -C debugdir

# 実行
./debugdir/src/jdim

# パッチの削除
rm -rf debugdir
git reset --hard master

hub コマンドを使う場合

hub checkout https://github.com/ma8ma/JDim/pull/59
meson debugdir
meson compile -C debugdir

# 実行
./debugdir/src/jdim

# パッチの削除
rm -rf debugdir
git checkout master
git branch -D idea-regex-implement-named-capture

[ma8ma]

Oniguruma利用時にregexのテストコードのビルドでposix regexと宣言が混ざりエラーが起きるようです

https://github.com/ma8ma/JDim/pull/59/checks?check_run_id=3645171604

In file included from /usr/include/gtest/internal/gtest-port.h:380,
                 from /usr/include/gtest/internal/gtest-internal.h:40,
                 from /usr/include/gtest/gtest.h:62,
                 from ../test/gtest_jdlib_jdregex.cpp:5:
/usr/include/regex.h:478:34: error: conflicting declaration ‘typedef struct re_pattern_buffer regex_t’
  478 | typedef struct re_pattern_buffer regex_t;
      |                                  ^~~~~~~
In file included from ../src/jdlib/jdregex.h:15,
                 from ../test/gtest_jdlib_jdregex.cpp:3:
/usr/include/onigposix.h:87:3: note: previous declaration as ‘typedef struct regex_t regex_t’
   87 | } regex_t;
      |   ^~~~~~~
In file included from /usr/include/gtest/internal/gtest-port.h:380,
                 from /usr/include/gtest/internal/gtest-internal.h:40,
                 from /usr/include/gtest/gtest.h:62,
                 from ../test/gtest_jdlib_jdregex.cpp:5:
/usr/include/regex.h:521:3: error: conflicting declaration ‘typedef struct regmatch_t regmatch_t’
  521 | } regmatch_t;
      |   ^~~~~~~~~~
In file included from ../src/jdlib/jdregex.h:15,
                 from ../test/gtest_jdlib_jdregex.cpp:3:
/usr/include/onigposix.h:80:3: note: previous declaration as ‘typedef struct regmatch_t regmatch_t’
   80 | } regmatch_t;
      |   ^~~~~~~~~~
In file included from /usr/include/gtest/internal/gtest-port.h:380,
                 from /usr/include/gtest/internal/gtest-internal.h:40,
                 from /usr/include/gtest/gtest.h:62,
                 from ../test/gtest_jdlib_jdregex.cpp:5:
/usr/include/regex.h:643:12: error: conflicting declaration of C function ‘int regexec(const regex_t*, const char*, size_t, regmatch_t*, int)’
  643 | extern int regexec (const regex_t *_Restrict_ __preg,
      |            ^~~~~~~
In file included from ../src/jdlib/jdregex.h:15,
                 from ../test/gtest_jdlib_jdregex.cpp:3:
/usr/include/onigposix.h:162:20: note: previous declaration ‘int regexec(regex_t*, const char*, size_t, regmatch_t*, int)’
  162 | ONIG_EXTERN int    regexec P_((regex_t* reg, const char* str, size_t nmatch, regmatch_t* matches, int options));
      |                    ^~~~~~~

[ma8ma]

clang-5.0 のコンパイルエラーのまとめ

• 現状の実装(noexcept)ではclang-5.0以外のjobはビルドに成功する
• noexcept(bool式)を使うとclangのjobで成功失敗が分かれる
• noexceptの使用をやめるとすべてのjobでビルドに成功する

[ma8ma]

テストの結果、現バージョン0.6.0でのマージを断念し動作環境が更新される0.7.0リリース以降に上流へマージしたいと思います。