Skip to content

Reformat C++ code to follow consistent style conventions#2

Merged
Fury7425 merged 1 commit into
mainfrom
claude/keen-noether-CfrlY
May 24, 2026
Merged

Reformat C++ code to follow consistent style conventions#2
Fury7425 merged 1 commit into
mainfrom
claude/keen-noether-CfrlY

Conversation

@Fury7425
Copy link
Copy Markdown
Owner

Summary

Applied comprehensive code formatting across C++ implementation files to enforce consistent style conventions throughout the codebase. This is a pure formatting change with no functional modifications.

Key Changes

  • Namespace formatting: Converted multi-line namespace declarations to single-line format (namespace yip { instead of namespace yip\n{)
  • Indentation: Removed excessive indentation within anonymous namespaces and implementation namespaces, aligning code to project conventions
  • Brace placement: Standardized opening braces to follow K&R style (same line for control structures, new line for functions)
  • Spacing: Normalized spacing around operators, function parameters, and struct initializers
  • Line continuations: Reformatted long function signatures and initializer lists for readability
  • Struct/enum formatting: Aligned struct and enum member declarations with consistent indentation

Files Modified

  • IndicatorWindow.xaml.cpp — Major formatting pass on composition layer, state machine, and event handlers
  • SettingsDialog.xaml.cpp — Namespace and method formatting
  • MainWindow.xaml.cpp — Namespace and using declarations
  • App.xaml.cpp — Namespace and lambda formatting
  • AudioCoreInterop.cpp — Class and function formatting
  • Settings.cpp — Namespace and function formatting
  • IndicatorPersistence.cpp — Namespace and function formatting
  • Markers.cpp — Namespace and function formatting
  • main.cpp — Function signature formatting
  • Header files (.h) — Struct, enum, and namespace formatting

Implementation Details

All changes maintain the existing code logic and behavior. The formatting aligns with C++20 conventions and the project's established style guidelines for namespace declarations, indentation depth, and brace placement. No functional changes, API modifications, or behavioral alterations were introduced.

https://claude.ai/code/session_015WKywfjPwRt1v9gGXqZN8o

Run clang-format -i against yip-app and vst-host using the existing
.clang-format config (LLVM-based, C++20, 110-col, 4-space indent).
All 17 files now pass --dry-run --Werror clean.

https://claude.ai/code/session_015WKywfjPwRt1v9gGXqZN8o
@Fury7425 Fury7425 merged commit 1b5bd1b into main May 24, 2026
2 of 3 checks passed
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