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

画像が見つからないときは常に警告する #806

Merged
merged 1 commit into from
Jul 9, 2017

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Jul 8, 2017

cf #803

image, icon, indepimageにおいて、画像が見つからないときは無言ではなく必ずwarnで警告するようにした。
また、メッセージがno such image: ID だと挙動として変なので、元のに合わせて image not bound: ID で揃えた。

@kmuto
Copy link
Owner Author

kmuto commented Jul 8, 2017

副作用はないと思いますが、いちおうご確認いただけますか。 @takahashim

@kmuto
Copy link
Owner Author

kmuto commented Jul 8, 2017

パッチでは、icon, indepimageで見つからないときの挙動も変更し、無視したり漫然とpreにするのではなく、image_dummy相当と同じことをするようにしています。
latexのinline_iconでverbにしているのが微妙かもですが、本来画像は全部あるのが適正のはずなので。

Copy link
Collaborator

@takahashim takahashim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ざっと見た感じでは問題ないかと思います

@kmuto
Copy link
Owner Author

kmuto commented Jul 9, 2017

ありがとうございます、マージします。

@kmuto kmuto merged commit 331aa87 into master Jul 9, 2017
@kmuto kmuto deleted the warnindepimage branch July 9, 2017 03:16
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