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 format #136

Closed
wants to merge 12 commits into from
Closed

Fix format #136

wants to merge 12 commits into from

Conversation

hzhuang1
Copy link

Fix some whitespace issues that are reported by check_format.sh.

Remove unnecessary whitespace that is recognized by check_format.sh.

$tools/check_format.sh -v

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
Remove unnecessary whitespace that is recognized by check_format.sh.

$./tools/check_format.sh -v

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
Remove unnecessary whitespace that is recognized by check_format.sh.

$./tools/check_format.sh -v

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
Remove unnecessary whitespace that is recognized by check_format.sh.

$./tools/check_format.sh -v

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
Remove unnecessary whitespace that is recognized by check_format.sh.

./tools/check_format.sh -v

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
Remove unnecessary whitespace that is recognized by check_format.sh.

$./tools/check_format.sh -v

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
Remove unnecessary whitespace that is recognized by check_format.sh.

$./tools/check_format.sh -v

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
Remove unnecessary whitespace that is recognized by check_format.sh.

$./tools/check_format.sh -v

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
Remove unnecessary whitespace that is recognized by check_format.sh.

$./tools/check_format.sh -v

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
Remove unnecessary whitespace that is recognized by check_format.sh.

$./tools/check_format.sh -v

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
Remove unnecessary whitespace that is recognized by tools/check_format.sh.

$./tools/check_format.sh -v

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
Remove unnecessary whitespace that is recognized by tools/check_format.sh.

$./tools/check_format.sh -v

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
@pablodelara
Copy link
Contributor

Hi @hzhuang1. Thanks for this PR. Could you let me know which "indent" version do you have? I don't see these formatting issues on my side.

@hzhuang1
Copy link
Author

Screenshot 2024-03-27 at 22 34 27

@pablodelara
Copy link
Contributor

Thanks for the info. It looks like there is a mismatch between your version (2.2.13) and the version installed on the CI machine (2.2.12), which has Ubuntu 22.04. I'll figure out a way to have both aligned.

@pablodelara
Copy link
Contributor

HI @hzhuang1. We are transitioning to clang-format, so this won't matter anymore. I will let you know when this is done. Thanks!

@pablodelara
Copy link
Contributor

@hzhuang1, I'm going to close this issue, as we are using now clang-format for coding style.

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