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

Fix -Wformat-security #2

Closed
wants to merge 1 commit into from
Closed

Conversation

kenji-miyake
Copy link

@kenji-miyake kenji-miyake commented Aug 15, 2021

Clangビルドエラーの修正です。
これはupstreamに出しても良いかも?
(型がイマイチ分かっておらず、対策がこれで良いのか若干怪しいです。)

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
@h-ohta
Copy link

h-ohta commented Aug 16, 2021

これは小さい変更なのでupstreamに出してもすぐ見てくれそうですね

@kenji-miyake
Copy link
Author

ros-drivers#170 でPRを出しました。これは閉じて、upstreamが取り込んでくれたらマージする感じが良いと思っています。

@kenji-miyake kenji-miyake deleted the fix-for-clang-build branch September 12, 2021 15:40
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