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

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

kobkaz
Copy link
Contributor

@kobkaz kobkaz commented Jan 4, 2024

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

@kobkaz kobkaz self-assigned this Jan 4, 2024
@@ -117,7 +117,7 @@ pub struct FieldIter<'a> {
}

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

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

@@ -131,10 +131,11 @@ fn build_field_schema(
obs: tlmdb::OnboardSoftwareInfo,
field: &tlmdb::Field,
bit_range: Range<usize>,
) -> Result<(&str, FieldSchema)> {
) -> Result<(&str, &tlmdb::DisplayInfo, FieldSchema)> {
Copy link

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

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 kobkaz force-pushed the display_info branch 2 times, most recently from fdc6796 to cb67b15 Compare January 4, 2024 09:52
@kobkaz kobkaz requested a review from sksat as a code owner January 4, 2024 09:52
@kobkaz kobkaz changed the title (WIP) Display info DisplayInfo の format を用いて devtoolsでの表示を指定する Jan 5, 2024
@kobkaz kobkaz force-pushed the display_info branch 3 times, most recently from 17bd1ed to 84bbb2c Compare January 10, 2024 08:02
@kobkaz kobkaz force-pushed the display_info branch 2 times, most recently from 590607f to 934f39d Compare January 19, 2024 00:35
@kobkaz
Copy link
Contributor Author

kobkaz commented Jan 19, 2024

ビルドこれでいいのか感はあるので @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