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

//beginchild, //endchildを使った自動判定による箇条書き子要素指示 #1497

Merged
merged 13 commits into from
Sep 28, 2020

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Apr 13, 2020

#1492 からの変形版

  • //beginchild//endchildの対にした。名前はもうちょっと考えたほうがいいか? 複雑になると構造がわかりにくい感じ。挙動としては無意味でもメモ用に引数をとるようにする?
  • Compilerでは箇条書きが終わったらビルダの@previous_last_typeに箇条書きの種類名を入れる。ただ、この変数をnilにするタイミングがないので、1回箇条書きを使いさえすれば以降のどこでも意図しない構造が作れてしまう。

  • //beginchild//endchild は0個または1個のオプションを取れるようにした。オプションはコメント用(たとえば//beginchild[番号箇条書き内始め])で、挙動には影響しない。
  • //begin,//endの対応関係を見るようにしているので、いちおう「意図しない構造」は発生しづらいと思う。

@kmuto kmuto mentioned this pull request Jun 12, 2020
@takahashim
Copy link
Collaborator

とりあえずconflictしていたのを修正しました。

@kmuto
Copy link
Owner Author

kmuto commented Jun 24, 2020

ありがとうございます!

@kmuto
Copy link
Owner Author

kmuto commented Sep 23, 2020

@takahashim
ブロック要素のほうは入れ子できるようにするとして、箇条書きのほうはあまりどうにもならない気がするので、結局こういう前後指示命令を入れていくぶん無理矢理対処するしかないのではと思います。

実験的にでも入れることは禍根になりそうでしょうかね…。ただ暗黙記法は増やしたくないし、インデントでの区別は嫌だし、箇条書きの性質的に//{なものも使えないですし。

@takahashim
Copy link
Collaborator

これですが、あまり綺麗な構文が思いつかないので(冗長に書いてもいいならブロックの入れ子で表現できそうですが、現状の箇条書きに比べると圧倒的に見通しが悪そう)、将来的に廃止することになった場合壊れても構わないのであれば入れても良いかも、と思いました。

@takahashim
Copy link
Collaborator

takahashim commented Sep 26, 2020

あー、でもコメントを取る機能はやめた方がいいかも、と思いました。他の記法と異なりすぎるので(コメント以外に使いたくなっても困るし)。

@kmuto
Copy link
Owner Author

kmuto commented Sep 27, 2020

なるほど、了解です。まぁ#@#でも入れとけという話ですしね。

@kmuto
Copy link
Owner Author

kmuto commented Sep 27, 2020

なんとなくconflictが出そうな気がするので、#1558 の後にマージしようかと思います。
macOSビルドはハイライトテストのところでなんか変ですね。たまにrougeかpygmentsの結果が一致しないことがありますな…。

@takahashim
Copy link
Collaborator

conflict解消してみました

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