Skip to content
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

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

Merged
merged 1 commit into from
Jun 30, 2022

Conversation

akinomyoga
Copy link
Member

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

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

@akinomyoga
Copy link
Member Author

akinomyoga commented Jun 5, 2022

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

@faithandbrave
Copy link
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