[Question] tokens: Next/Prev and reserved keywords about go-yaml HOT 2 CLOSED

1. 現在、Tokens = []*Token で、添え字アクセスしつつ、Next/Prevを使うことをしてますが、これってNext/Prevだけでlinked listを作ればよくないですか？

token.Tokens は、 Lexer から返してあげた tokens を iteration するときに for range を使わせてあげたいっていうところからきてまして、内部で Next , Prev をもってるのは実装上の都合だったので、公開している I/F を変えない範囲で ( Parser の内部処理など ) は変更しても良いかなと思っています。

ただ、 Next , Prev を利用するようにすると、 token.Tokens に対して好きに token を追加削除した後に Parser に渡すとおかしくなってしまうので、このあたりをケアしてあげるような API を追加 ( Tokens.Insert や Tokens.Remove を足すとか ) する必要があるなと思っていました。

2. reserved tokensは一回作って使い回していますが、これって複数のgoroutineでアクセスするときとか、filteredTokensとかを作る時にNext/Prevとかの値が上書きされておかしなことになりませんか？reservedも動的に生成するようにしていいですか？

あれ、 reserved なやつは、 token 生成用の function map で管理しているので、都度生成するようになっていると思いますが、そういうことではなくでしょうか

from go-yaml.

2は 失礼しました！読み間違えてました！

from go-yaml.