-
Notifications
You must be signed in to change notification settings - Fork 215
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
use IndexBuilder instead of *Index.parse() #1384
Conversation
先に |
方針よさそうなので進めていただくことに |
4ead5b7
to
76ec859
Compare
6e4cca6
to
b90cdec
Compare
b90cdec
to
cc31774
Compare
とりあえず動くようになった(けど遅そう) |
と思ったけどまだダメそう… |
これでだいたい直ったはず |
IndexBuilder#bind で |
なるほど、足しておきます |
make indexes with this builder, instead of `Index.parse()`
4b3993c
to
c5475b7
Compare
ありがとうございます、私の抱えるプロジェクト範囲だと特に問題はないようです。とりあえず遅くなったというのも感じないような。 |
遅かったのは同じファイルに対してIndexBuilderが何度も走っていたからでした。ここは整理したので、現状はそんなに遅くなっていない(といいなあ)、というところです |
だいたいこれで良さそうなので、あとでマージしてみます |
ReVIEW::Book::Index.parse
を全面的に置き換えるものです。インデックス専用のbuilderを作って何とかする作戦です。