Multi-line paste warning doesn't detect \r, only \n #11822

Open
opened 2026-01-31 02:58:25 +00:00 by claunia · 0 comments
Owner

Originally created by @DHowett on GitHub (Dec 16, 2020).

Certain shells will execute commands on \r as well as \n, so we should try to catch \r as well.

This should probably be tackled at the same time as paste filtering. I originally thought that we should detect newlines after filtering, or perhaps during filtering, but would result in us doing a "lot" of work in the case where the user declines the paste.

Filtering: #5821.

Originally created by @DHowett on GitHub (Dec 16, 2020). Certain shells will execute commands on \r as well as \n, so we should try to catch \r as well. This should probably be tackled at the same time as paste filtering. I originally thought that we should detect newlines _after_ filtering, or perhaps _during_ filtering, but would result in us doing a "lot" of work in the case where the user declines the paste. Filtering: #5821.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#11822