Skip to content

Refactor: Clean up and format FsCCplus source code#40

Merged
Hashory merged 2 commits intomainfrom
refactor/fs-ccplus-cleanup
Aug 23, 2025
Merged

Refactor: Clean up and format FsCCplus source code#40
Hashory merged 2 commits intomainfrom
refactor/fs-ccplus-cleanup

Conversation

@google-labs-jules
Copy link
Copy Markdown
Contributor

This pull request applies a series of refactoring and formatting improvements to the source code within the Source/Effects/FsCCplus directory, following the user's detailed instructions. The changes include removing obsolete comments, translating all Japanese comments to English, standardizing include guards to use #pragma once, protecting include blocks from reordering, and applying clang-format for consistent code style.

This update also addresses a correction for Fs_Entry.h, which was improperly modified in the initial attempt. The file has been restored and correctly refactored to ensure all necessary includes are present.

This commit refactors the C++ source code in the `Source/Effects/FsCCplus` directory.

The following changes were made:
- Removed unnecessary decorative and historical comments.
- Translated all Japanese comments to English for better readability and maintenance.
- Replaced traditional `#ifndef` include guards with `#pragma once` in header files.
- Wrapped `#include` blocks with `// clang-format off/on` to prevent reordering by formatters.
- Applied clang-format to the entire directory for consistent code style.

This commit also corrects a mistake in the initial refactoring of `Fs_Entry.h` where a necessary block of includes was removed instead of being uncommented and refactored.
@Hashory Hashory marked this pull request as ready for review August 23, 2025 05:57
@Hashory Hashory self-requested a review August 23, 2025 05:57
@Hashory Hashory merged commit e805ee0 into main Aug 23, 2025
2 checks passed
@Hashory Hashory deleted the refactor/fs-ccplus-cleanup branch August 23, 2025 05:59
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