Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid escape sequence \d #18

Closed
AlexMax opened this issue Dec 12, 2023 · 2 comments
Closed

Invalid escape sequence \d #18

AlexMax opened this issue Dec 12, 2023 · 2 comments

Comments

@AlexMax
Copy link

AlexMax commented Dec 12, 2023

I'm getting this message when attempting to run master with Python 3.12.1 on Windows

PS ...> binmerge.py
...\binmerge.py:122: SyntaxWarning: invalid escape sequence '\d'
  m = re.search('TRACK (\d+) ([^\s]*)', line)
...\binmerge.py:128: SyntaxWarning: invalid escape sequence '\d'
  m = re.search('INDEX (\d+) (\d+:\d+:\d+)', line)
...\binmerge.py:169: SyntaxWarning: invalid escape sequence '\d'
  m = re.match("(\d+):(\d+):(\d+)", stamp)
usage: binmerge.py [-h] [-l] [-v] [-s] [-o OUTDIR] cuefile basename
binmerge.py: error: the following arguments are required: cuefile, basename

Seems like some of these strings are missing r prefixes.

@putnam
Copy link
Owner

putnam commented Dec 12, 2023

Thanks for the report. Looks like 3.12 raised this warning, which was previously a (hidden) DeprecationWarning.

The app should run fine regardless, but I will fix those regex strings.

@putnam
Copy link
Owner

putnam commented Dec 12, 2023

Fixed in 5576322

@putnam putnam closed this as completed Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants