[PR #980] [MERGED] Remove trailing whitespaces #1728

New issue

Closed

opened 2026-05-05 08:00:27 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 08:00:27 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/debauchee/barrier/pull/980
Author: @whot
Created: 12/9/2020
Status: ✅ Merged
Merged: 12/9/2020
Merged by: @p12tic

Base: master ← Head: wip/strip-trailing-whitespace

📝 Commits (1)

0dbedd3 Remove trailing whitespaces

📊 Changes

476 files changed (+1541 additions, -1541 deletions)

View changed files

📝 CMakeLists.txt (+2 -2)
📝 ext/openssl/windows/x64/include/openssl/opensslconf.h (+1 -1)
📝 ext/openssl/windows/x86/include/openssl/opensslconf.h (+1 -1)
📝 src/CMakeLists.txt (+2 -2)
📝 src/cmd/CMakeLists.txt (+2 -2)
📝 src/cmd/barrierc/CMakeLists.txt (+2 -2)
📝 src/cmd/barrierc/MSWindowsClientTaskBarReceiver.cpp (+2 -2)
📝 src/cmd/barrierc/MSWindowsClientTaskBarReceiver.h (+2 -2)
📝 src/cmd/barrierc/OSXClientTaskBarReceiver.cpp (+2 -2)
📝 src/cmd/barrierc/OSXClientTaskBarReceiver.h (+2 -2)
📝 src/cmd/barrierc/XWindowsClientTaskBarReceiver.cpp (+2 -2)
📝 src/cmd/barrierc/XWindowsClientTaskBarReceiver.h (+2 -2)
📝 src/cmd/barrierc/barrierc.cpp (+4 -4)
📝 src/cmd/barrierc/resource.h (+1 -1)
📝 src/cmd/barrierd/barrierd.cpp (+2 -2)
📝 src/cmd/barriers/CMakeLists.txt (+2 -2)
📝 src/cmd/barriers/MSWindowsServerTaskBarReceiver.cpp (+2 -2)
📝 src/cmd/barriers/MSWindowsServerTaskBarReceiver.h (+2 -2)
📝 src/cmd/barriers/OSXServerTaskBarReceiver.cpp (+2 -2)
📝 src/cmd/barriers/OSXServerTaskBarReceiver.h (+2 -2)

...and 80 more files

📄 Description

Many tools strip trailing whitespaces by default, so after editing a file with
whitespace errors we end up with a bunch of unrelated hunks that need to be
reverted locally.

This could be fixed by configuring each tool to not do this (at least for the
barrier repo), or, simpler, we just sed the problem away and make barrier
whitespace-compliant.

sed commands run:

 sed -i 's/[ \t]\+$//' **/*.(cpp|h) **/*CMakeLists.txt

Verified with git diff --ignore-space-change, this shows the empty diff.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/debauchee/barrier/pull/980 **Author:** [@whot](https://github.com/whot) **Created:** 12/9/2020 **Status:** ✅ Merged **Merged:** 12/9/2020 **Merged by:** [@p12tic](https://github.com/p12tic) **Base:** `master` ← **Head:** `wip/strip-trailing-whitespace` --- ### 📝 Commits (1) - [`0dbedd3`](https://github.com/debauchee/barrier/commit/0dbedd31dc6e3d9a7d2dea3e330ac07c16f0d852) Remove trailing whitespaces ### 📊 Changes **476 files changed** (+1541 additions, -1541 deletions) <details> <summary>View changed files</summary> 📝 `CMakeLists.txt` (+2 -2) 📝 `ext/openssl/windows/x64/include/openssl/opensslconf.h` (+1 -1) 📝 `ext/openssl/windows/x86/include/openssl/opensslconf.h` (+1 -1) 📝 `src/CMakeLists.txt` (+2 -2) 📝 `src/cmd/CMakeLists.txt` (+2 -2) 📝 `src/cmd/barrierc/CMakeLists.txt` (+2 -2) 📝 `src/cmd/barrierc/MSWindowsClientTaskBarReceiver.cpp` (+2 -2) 📝 `src/cmd/barrierc/MSWindowsClientTaskBarReceiver.h` (+2 -2) 📝 `src/cmd/barrierc/OSXClientTaskBarReceiver.cpp` (+2 -2) 📝 `src/cmd/barrierc/OSXClientTaskBarReceiver.h` (+2 -2) 📝 `src/cmd/barrierc/XWindowsClientTaskBarReceiver.cpp` (+2 -2) 📝 `src/cmd/barrierc/XWindowsClientTaskBarReceiver.h` (+2 -2) 📝 `src/cmd/barrierc/barrierc.cpp` (+4 -4) 📝 `src/cmd/barrierc/resource.h` (+1 -1) 📝 `src/cmd/barrierd/barrierd.cpp` (+2 -2) 📝 `src/cmd/barriers/CMakeLists.txt` (+2 -2) 📝 `src/cmd/barriers/MSWindowsServerTaskBarReceiver.cpp` (+2 -2) 📝 `src/cmd/barriers/MSWindowsServerTaskBarReceiver.h` (+2 -2) 📝 `src/cmd/barriers/OSXServerTaskBarReceiver.cpp` (+2 -2) 📝 `src/cmd/barriers/OSXServerTaskBarReceiver.h` (+2 -2) _...and 80 more files_ </details> ### 📄 Description Many tools strip trailing whitespaces by default, so after editing a file with whitespace errors we end up with a bunch of unrelated hunks that need to be reverted locally. This could be fixed by configuring each tool to not do this (at least for the barrier repo), or, simpler, we just sed the problem away and make barrier whitespace-compliant. sed commands run: ``` sed -i 's/[ \t]\+$//' **/*.(cpp|h) **/*CMakeLists.txt ``` Verified with `git diff --ignore-space-change`, this shows the empty diff. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>