Skip to content

Code review: Apply clang-format and add .gitignore#5

Draft
Copilot wants to merge 6 commits intodevelopfrom
copilot/review-code
Draft

Code review: Apply clang-format and add .gitignore#5
Copilot wants to merge 6 commits intodevelopfrom
copilot/review-code

Conversation

Copy link

Copilot AI commented Jan 12, 2026

Comprehensive code review and formatting cleanup of the codebase.

Changes

  • Formatting: Applied clang-format to 28 source files for consistency with .clang-format and CODE_STYLE.md

    • Fixed line breaks, indentation, and spacing
    • Standardized multi-line statement formatting
  • Repository hygiene: Added .gitignore to exclude build artifacts, compilation databases, and IDE files

Example

Before:

wlan_emu_print(wlan_emu_log_level_dbg,
    "%s:%d: mac_str : %s rssi : %d noise : %d bitrate : %d\n",
    __func__, __LINE__, to_mac_str(heart_beat_data->mac, mac_str), heart_beat_data->rssi,
    heart_beat_data->noise, heart_beat_data->bitrate);

After:

wlan_emu_print(wlan_emu_log_level_dbg,
    "%s:%d: mac_str : %s rssi : %d noise : %d bitrate : %d\n", __func__, __LINE__,
    to_mac_str(heart_beat_data->mac, mac_str), heart_beat_data->rssi, heart_beat_data->noise,
    heart_beat_data->bitrate);

No functional changes. Static analysis and security scans completed with no issues.

Original prompt

review this code


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 5 commits January 12, 2026 12:20
Co-authored-by: gsathish86 <9947342+gsathish86@users.noreply.github.com>
Co-authored-by: gsathish86 <9947342+gsathish86@users.noreply.github.com>
Co-authored-by: gsathish86 <9947342+gsathish86@users.noreply.github.com>
Co-authored-by: gsathish86 <9947342+gsathish86@users.noreply.github.com>
Co-authored-by: gsathish86 <9947342+gsathish86@users.noreply.github.com>
Copilot AI changed the title [WIP] Review code for quality and performance Code review: Apply clang-format and add .gitignore Jan 12, 2026
Copilot AI requested a review from gsathish86 January 12, 2026 12:24
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