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

ブロック内等の「===」が見出し行としてHeadlineIndex.parseで処理される #121

Closed
takahashim opened this issue Apr 19, 2012 · 7 comments

Comments

@takahashim
Copy link
Collaborator

//emlist{
==============
test
==============
//}

といった記述の「==============」がHeadlineIndex.parseでは見出し行として認識され、しかもduplicate IDの警告が出ます(同じ見出しにみえるので)。

そうは言っても直すのは大変なのでwontfixですかねえ…

@kdmsnr
Copy link
Collaborator

kdmsnr commented May 2, 2012

\A=+のあとに=じゃないものが続くところを見出しとしてパースするとかはどうでしょう?

@takahashim
Copy link
Collaborator Author

さっきやっと上の指摘の意味が理解できたのですが、その場合はたとえば

//emlist{
=begin
コメント
=end
//}

でもマズイですよね…。それでもないよりはマシでしょうか。

@kdmsnr
Copy link
Collaborator

kdmsnr commented May 9, 2012

ぐぬぬ。

  • begin/endは無視する
  • == のあとに空白が必要

にするとかでしょうか。

@kmuto
Copy link
Owner

kmuto commented Nov 9, 2017

今は別の、より妙な状況になりますね。

  • 上記のemlistだけではエラー・警告にならず、正常に生成される。
  • hdタグをどこかで使ったときに顕在化し、エラーもERROR -- : r.re:3: error: private method warn' called for nil:NilClass`という助けにならないものしか出ない。
= section

//footnote[footnote_id][@<hd>{unknown}]

== subsection

//emlist[][rst]{
======
見出し
======
//}

=={unknown} hoge

unknown見出しの有無や、見出しの位置は関係なし。emlistの======を取ったり先頭スペースを入れたりすると正常になる。

@tk0miya
Copy link

tk0miya commented Nov 9, 2017

https://gist.github.com/tk0miya/ca9ca030d50b4c9ae67d6bb6ed8aad71
の内容を vvakame/review docker イメージで処理すると、以下の warning が出ます。
@<hd> を削除すると warning はでなくなります。

docker run -it --rm -v `pwd`:/work vvakame/review bash -c "cd /work; review-webmaker config.yml"
warning: duplicate ID: subsection||||
content.re:3: error: unknown hd: unknown

@kmuto
Copy link
Owner

kmuto commented Nov 9, 2017

これ、HeadlineIndexでblockかどうかを判断してないのがまずいだけですね。手元で修正して今テスト中。

@kmuto
Copy link
Owner

kmuto commented Nov 11, 2017

#871にて修正

@kmuto kmuto closed this as completed Nov 11, 2017
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

No branches or pull requests

4 participants