Fix invalid escape sequences, noisy since py3.12, deprecated since py3.6

[zer0def]

Fixes invalid escape sequences, which were deprecated since py3.6 and started raising SyntaxWarnings since py3.12, example follows.

python: stdout/stderr (?): /home/user/.weechat/python/twitch.py:354: SyntaxWarning: invalid escape sequence '\s'
python: stdout/stderr (?):   bn=tags['ban-reason'].replace('\s',' ')
python: stdout/stderr (?): /home/user/.weechat/python/twitch.py:362: SyntaxWarning: invalid escape sequence '\s'
python: stdout/stderr (?):   bn=tags['ban-reason'].replace('\s',' ')
python: stdout/stderr (?): /home/user/.weechat/python/twitch.py:459: SyntaxWarning: invalid escape sequence '\s'
python: stdout/stderr (?):   msg = tags['system-msg'].replace('\s',' ')

Upstream references:

• Reject invalid escape sequences (and octal escape sequences) in bytes and Unicode strings python/cpython#98401
• https://docs.python.org/3/whatsnew/3.12.html#other-language-changes

[mumixam]

Thanks for reporting.
I'm running a older version of python3 so i havent noticed this yet.

I went ahead and tested with 3.12 but i made of the change of changing replace('\s',' ') to replace(r'\s',' ') it seems to fix the issue. Looks like twitch doesn't announce the reason for the ban anymore via irc tags anyway but i tested in my own channel banning a user that wasn't in the chat. Did you happen to test with replace(r'\s',' ') and have issues?

[zer0def]

I have, it made the warnings go away, but no longer performed the substitution. At best, it's the same kind of no-op, as it is before this change.

$ python
Python 3.13.1 (main, Dec  4 2024, 18:05:56) [GCC 14.2.1 20240910] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import re; a = '\t'
>>> ' '.replace(r' ', 'a')
'a'
>>> a.replace(r'\s', 'a')
'\t'
>>> a.replace(r'\\s', 'a')
'\t'
>>> re.sub(r'\s', 'a', a)
'a'

[zer0def]

Updated handling on the system-msg tag, as it turns out to have it's string delimited by raw \s substring, not a whitespace matcher.