簡易パースをしてnonumを採番スキップする #554
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Apr 14, 2016
Merged
| @@ -35,6 +33,7 @@ def initialize(book, number, name, path, io = nil) | |||
| end | |||
| if !@content && @path && File.exist?(@path) | |||
| @content = File.read(@path).sub(/\A\xEF\xBB\xBF/u, '') | |||
| @number = nil if check_header == "nonum" | |||
There was a problem hiding this comment.
このcheck_headerってChapterのインスタンスを作るときに1回だけチェックしているようですが、[nonum]が章・節・小節等のいろんなところに出たり出なかったりする場合でもこのコードで問題ないんでしょうか…？
There was a problem hiding this comment.
そうか、だめですねこれでは…
だいぶ厄介な気がしてきましたぞ…
There was a problem hiding this comment.
いや、大丈夫っぽいです。
ここでやっていることは
- ともかく先頭の見出しを探して（最初のレベルが1とは限らない）、
- nonum, nodisp, notocだったらそのファイルは章として採番しない（章カウンタをスキップしたい）ので、このChapterオブジェクトのnumber参照時をnilにする
という目的です。
本文のほうは、HTMLBuilderであればheadline→sec_counterで採番していますが、nonumはnonum_beginに落ちてsec_counterを使わないので採番増加も起きません（ほかのno*も同様）。
There was a problem hiding this comment.
名前をfind_first_header_option にでもするか
There was a problem hiding this comment.
なるほど、ここのロジックとしては理解できました。先頭が章レベルの見出しじゃない場合があるのが微妙な挙動につながるわけですね。うーむ。
|
テスト用に https://github.com/reviewml/review-samples/tree/master/nonum を作ってみました。 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#550 のパッチをさらに進めて、nonumの採番を飛ばすようにしてみました。
Chapter#initializeでコンテンツを読み込んでたので、ここでヘッダを探して（Chapter#check_header）見出しタグのtag部がnonumだったらnumberをnilにします。
……で、せっかくPEG化も考えている中、check_headerがまたcompiler由来の正規表現コードになっています。簡易としてヘッダとブロック以外は考えずに解析を回しています。