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

Testing clang-format CI #3

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Testing clang-format CI #3

wants to merge 3 commits into from

Conversation

antangelo
Copy link
Owner

No description provided.

@antangelo antangelo force-pushed the ci/clang-format-test branch 3 times, most recently from 4dfe402 to d29e9b9 Compare July 29, 2023 05:55
@github-actions
Copy link

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-format reports: 1 file(s) not formatted
  • softmmu/vl.c

Have any feedback or feature suggestions? Share it here.

@antangelo antangelo force-pushed the ci/clang-format-test branch from fe87575 to e950282 Compare January 2, 2025 23:54
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Cpp-linter Review

Used clang-format v16.0.6

Click here for the full clang-format patch
diff --git a/softmmu/vl.c b/softmmu/vl.c
index 447eee7..5582986 100644
--- a/softmmu/vl.c
+++ b/softmmu/vl.c
@@ -2820 +2820,2 @@ void qemu_init(int argc, char **argv)
-    } if (g_config.general.show_welcome) {
+    }
+    if (g_config.general.show_welcome) {
@@ -2835 +2836 @@ void qemu_init(int argc, char **argv)
-        fake_argv[fake_argc++] = strdup("-m");
+    fake_argv[fake_argc++] = strdup("-m");

Have any feedback or feature suggestions? Share it here.


if (flashrom_override_path) {
assert(!xemu_check_file(flashrom_override_path));
} if (g_config.general.show_welcome) {
Copy link

Choose a reason for hiding this comment

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

clang-format suggestion

Suggested change
} if (g_config.general.show_welcome) {
}
if (g_config.general.show_welcome) {

@@ -2820,7 +2832,7 @@
}

int mem = ((int)g_config.sys.mem_limit + 1) * 64;
fake_argv[fake_argc++] = strdup("-m");
fake_argv[fake_argc++] = strdup("-m");
Copy link

Choose a reason for hiding this comment

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

clang-format suggestion

Suggested change
fake_argv[fake_argc++] = strdup("-m");
fake_argv[fake_argc++] = strdup("-m");

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.

1 participant