search

kvirc / KVIrc

The KVIrc IRC Client
http://www.kvirc.net/
GNU General Public License v2.0
232 stars 76 forks source link

Strip leading spaces on command; Fix #2626 #2627

Closed ctrlaltca closed 3 months ago

ctrlaltca commented 3 months ago

KVIrc used to strip any space at the beginning of a message/command. This behavior was reported as a bug in #1895 and changed in eed2e4dd7779d38e336834c7054e5be67b97094e. While it's good to keep any space at the beginning of a text message, it's good to strip them if the message contains a command.

DarthGandalf commented 3 months ago

To clarify: this affects both user-friendly and KVS modes? Should it?

Code-wise, this seems fine.

ctrlaltca commented 3 months ago

To clarify: this affects both user-friendly and KVS modes? Should it?

Yes, it affects both user-friendly and KVS modes. I guess it should, seems like a regression introduced in eed2e4dd7779d38e336834c7054e5be67b97094e to fix another usecase.