Skip to content

定義語自動リンク: プロセッサ markdown_to_html.defined_words を追加#79

Merged
faithandbrave merged 1 commit into
cpprefjp:masterfrom
akinomyoga:defined_words
Jun 30, 2022
Merged

定義語自動リンク: プロセッサ markdown_to_html.defined_words を追加#79
faithandbrave merged 1 commit into
cpprefjp:masterfrom
akinomyoga:defined_words

Conversation

@akinomyoga

Copy link
Copy Markdown
Member

cpprefjp/site#977 に関連する変更です。

cpprefjp/markdown_to_html#5 への参照 (.gitmodules) がありますので、そちらが確定するまで Draft PR とします。

@akinomyoga

akinomyoga commented Jun 5, 2022

Copy link
Copy Markdown
Member Author

実装の詳細になりますが、hrefs, global_qualify_list, global_defined_words の3つの引数の組を何度も子関数に直接渡しているので、これらを一つのオブジェクト resources={hrefs = ..., global_qualify_list = ..., global_defined_words = ...} にまとめてしまった方が見通しが良いかもしれません。現時点では、元々の実装(2つの引数)に倣って、新しい引数global_defined_wordsを加えて毎回3つの引数を直接渡しています。

@faithandbrave

Copy link
Copy Markdown
Member

すいません、間違えました…。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants