Skip to content

Comments

Trim trailing tabs as whitespace in trimtrailingspace#42

Merged
rainman74 merged 2 commits intomainfrom
codex/fix-access-violation-in-trim-trailing-spaces
Feb 17, 2026
Merged

Trim trailing tabs as whitespace in trimtrailingspace#42
rainman74 merged 2 commits intomainfrom
codex/fix-access-violation-in-trim-trailing-spaces

Conversation

@rainman74
Copy link
Owner

Motivation

  • After previous safety fixes removed a crash path, trailing whitespace still remained when lines ended with tabs, so trimtrailingspace must treat \t as removable whitespace in addition to space.

Description

  • Updated SRC/NPPTextFX.cpp in trimtrailingspace to change the backward-trim predicate from checking only ' ' to checking (' ' || '\t'), so trailing horizontal tabs are removed along with spaces.

Testing

  • Applied and verified the one-line change with nl -ba SRC/NPPTextFX.cpp | sed -n '3672,3681p', git diff --stat, and committed the change; those checks succeeded.
  • No Notepad++/Scintilla runtime tests were available in this environment, so behavioral runtime validation was not performed.

Codex Task

@rainman74 rainman74 merged commit 37f8e45 into main Feb 17, 2026
@rainman74 rainman74 deleted the codex/fix-access-violation-in-trim-trailing-spaces branch February 17, 2026 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant