search

H1rono / traq-bot-http-rs

traQ BOTのHTTPリクエストパーサー
https://crates.io/crates/traq-bot-http
MIT License
4 stars 0 forks source link

`RequestParser::parse`周り修正 #133

Closed H1rono closed 2 months ago

H1rono commented 7 months ago

機能追加の動機 example複雑すぎる https://github.com/H1rono/traq-bot-http-rs/blob/33f424916e26838cc5d892d84b13e97afbe0a4dc/src/parser.rs#L129-L149

Iterator<...>ではなくIntoIterator<...>にした方が扱いやすい気がするのと、K, Vは参照なしで受け取った方が広い https://github.com/H1rono/traq-bot-http-rs/blob/33f424916e26838cc5d892d84b13e97afbe0a4dc/src/parser.rs#L66-L68 https://github.com/H1rono/traq-bot-http-rs/blob/33f424916e26838cc5d892d84b13e97afbe0a4dc/src/parser.rs#L152-L154

望ましい解決方法 やるだけ

他に検討した代替案

補足説明 後方互換性が怪しい

H1rono commented 7 months ago

149

H1rono commented 3 months ago

K, Vは参照なしで受け取った方が広い

AsRef<[u8]> for &strがないのでこうすると壊れる