Skip to content

Correct whitespace#28

Merged
teddych merged 2 commits intoteddych:masterfrom
ekohl:fix-whitespace
Feb 1, 2026
Merged

Correct whitespace#28
teddych merged 2 commits intoteddych:masterfrom
ekohl:fix-whitespace

Conversation

@ekohl
Copy link
Contributor

@ekohl ekohl commented Jan 13, 2026

See individual commits for details.

@teddych
Copy link
Owner

teddych commented Jan 19, 2026

Thanks for this work.
In Utils/Network.cpp there went something wrong. The indention of the case with {} is different than al the rest of the project.

@teddych teddych marked this pull request as draft January 19, 2026 12:36
ekohl added 2 commits January 25, 2026 11:34
Generated by:

    sed -i 's/ \t/\t/g' $(rg --type cpp --files-with-matches ' \t')
@ekohl
Copy link
Contributor Author

ekohl commented Jan 25, 2026

In Utils/Network.cpp there went something wrong. The indention of the case with {} is different than al the rest of the project.

It took me a moment to understand what you meant. I think I got it, but please consider using inline comments to point out issues.

@ekohl ekohl marked this pull request as ready for review January 25, 2026 10:42
@teddych teddych merged commit 57d14ec into teddych:master Feb 1, 2026
6 checks passed
@ekohl ekohl deleted the fix-whitespace branch February 1, 2026 21:29
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