Skip to content

Conversation

@Amonare
Copy link
Collaborator

@Amonare Amonare commented Jul 18, 2025

IMPORTANT: The commits do not change functionality of the code!

For more information of the work done, please check individual commits and their message:

  • zephyrdoom: Clean up build and configuration files;
  • zephyrdoom, config: Clean implementation sources;
  • zephyrdoom, src: Clean implementation sources;

@Amonare Amonare requested a review from zippko July 18, 2025 12:24
@Amonare Amonare force-pushed the zephyrdoom-code-cleanup branch from a59e0ed to 5c0f1b6 Compare July 18, 2025 13:52
@Amonare Amonare marked this pull request as ready for review July 18, 2025 13:53
@Amonare Amonare force-pushed the zephyrdoom-code-cleanup branch from 5c0f1b6 to 4c6ca2d Compare July 18, 2025 14:03
@Amonare Amonare force-pushed the zephyrdoom-code-cleanup branch 3 times, most recently from 26edd56 to ca81819 Compare August 4, 2025 09:50
@Amonare Amonare force-pushed the zephyrdoom-code-cleanup branch 2 times, most recently from f77f256 to bbf142f Compare August 20, 2025 10:13
@Amonare Amonare changed the title Source code cleanup of zephyrdoom directory zephyr-doom: Source code cleanup of zephyrdoom directory Aug 21, 2025
@Amonare Amonare changed the title zephyr-doom: Source code cleanup of zephyrdoom directory zephyrdoom: Source code cleanup Aug 21, 2025
@Amonare Amonare requested a review from zippko August 21, 2025 10:30
@Amonare Amonare force-pushed the zephyrdoom-code-cleanup branch 2 times, most recently from bf5dc2d to c749e0d Compare August 22, 2025 10:41
The commit contains the following changes:
- Fix of comments structure;
- Unification of whitespaces compared to other similar files in the
  repository;
- Removal of redundant/deprecated comments;
The commit contains the following changes:
- Clean up of redundant/deprecated comments;
- Clean up of redundant whitespaces and new lines;
- Fix of spacing within lines;
- Unify C comments to C89 and C98 code structure (/**/ instead of //);
- Fix too long lines;
- Unify comments among files;

The source code structure has been unified using default auto formatter of
C code which is up to date with recent standards.

In Visual Studio code if not enabled by default, the key combination is
Shift + Alt + F

Some exceptions have been applied for better readability due to Doxygen
and ifndef/ifdef directives available in the code.
IMPORTANT: The commit does not change functionality of the code!

The commit contains the following changes:
- Clean up of redundant/deprecated comments;
- Clean up of redundant whitespaces and new lines;
- Fix of spacing within lines;
- Unify C comments to C89 and C98 code structure (/**/ instead of //);
- Fix too long lines;
- Unify comments among files;
- Unify TODO sections within the code;
- Remove deprecated sections of code;
- Fix combining and dividing of conditions and loops;
- Fix typos;
- Fix various definitions of functions;

Additionally, the source code structure has been unified using default auto
formatter of C code which is up to date with recent standards.

In Visual Studio code if not enabled by default, the key combination is
Shift + Alt + F

Some exceptions have been applied for better readability due to Doxygen,
Chocolate Doom custom parser documentation and various directives available in
the code.
@Amonare Amonare force-pushed the zephyrdoom-code-cleanup branch from c749e0d to 0cefc14 Compare August 22, 2025 10:43
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.

3 participants