Skip to content

DisplayInfo の format を用いて devtoolsでの表示を指定する#58

Open
KOBAYASHI Kazuhiro (kobkaz) wants to merge 4 commits intomainfrom
display_info
Open

DisplayInfo の format を用いて devtoolsでの表示を指定する#58
KOBAYASHI Kazuhiro (kobkaz) wants to merge 4 commits intomainfrom
display_info

Conversation

@kobkaz
Copy link
Copy Markdown
Member

@kobkaz KOBAYASHI Kazuhiro (kobkaz) commented Jan 4, 2024

#47 display info のうちformatに対応する
interpolatorの仕様上位置引数は使えなさそうなので、暗黙に valueという変数として扱う
display info の format は {value:#0X} のようになる


impl<'a> Iterator for FieldIter<'a> {
type Item = Result<(&'a str, FieldSchema)>;
type Item = Result<(&'a str, &'a tlmdb::DisplayInfo, FieldSchema)>;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [clippy] reported by reviewdog 🐶
cannot find type DisplayInfo in module tlmdb

field: &tlmdb::Field,
bit_range: Range<usize>,
) -> Result<(&str, FieldSchema)> {
) -> Result<(&str, &tlmdb::DisplayInfo, FieldSchema)> {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [clippy] reported by reviewdog 🐶
cannot find type DisplayInfo in module tlmdb

let converter = build_integral_converter(&field.conversion_info);
Ok((
&field.name,
&field.display_info,
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [clippy] reported by reviewdog 🐶
no field display_info on type &tlmcmddb::tlm::Field

@kobkaz KOBAYASHI Kazuhiro (kobkaz) force-pushed the display_info branch 2 times, most recently from fdc6796 to cb67b15 Compare January 4, 2024 09:52
@kobkaz KOBAYASHI Kazuhiro (kobkaz) changed the title (WIP) Display info DisplayInfo の format を用いて devtoolsでの表示を指定する Jan 5, 2024
@kobkaz KOBAYASHI Kazuhiro (kobkaz) force-pushed the display_info branch 3 times, most recently from 17bd1ed to 84bbb2c Compare January 10, 2024 08:02
@kobkaz KOBAYASHI Kazuhiro (kobkaz) force-pushed the display_info branch 2 times, most recently from 590607f to 934f39d Compare January 19, 2024 00:35
@kobkaz
Copy link
Copy Markdown
Member Author

ビルドこれでいいのか感はあるので sksat (@sksat) に見てほしい

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