Improved Error message for failing re expressions

[lingster]

BPO	16007
Nosy	@benjaminp, @ezio-melotti, @serhiy-storchaka
Superseder	bpo-22578: Add additional attributes to re.error
Files	re.py: amend regex.

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2016-10-16.09:02:54.839>
created_at = <Date 2012-09-23.16:14:27.264>
labels = ['expert-regex', 'type-feature']
title = 'Improved Error message for failing re expressions'
updated_at = <Date 2016-10-16.09:02:54.837>
user = 'https://bugs.python.org/lingster'

bugs.python.org fields:

activity = <Date 2016-10-16.09:02:54.837>
actor = 'serhiy.storchaka'
assignee = 'none'
closed = True
closed_date = <Date 2016-10-16.09:02:54.839>
closer = 'serhiy.storchaka'
components = ['Regular Expressions']
creation = <Date 2012-09-23.16:14:27.264>
creator = 'lingster'
dependencies = []
files = ['27266']
hgrepos = []
issue_num = 16007
keywords = []
message_count = 6.0
messages = ['171051', '171054', '171055', '171702', '227072', '278754']
nosy_count = 8.0
nosy_names = ['benjamin.peterson', 'ezio.melotti', 'mrabarnett', 'BreamoreBoy', 'Ramchandra Apte', 'serhiy.storchaka', 'lingster', 'Wael.Al.Jishi']
pr_nums = []
priority = 'normal'
resolution = 'out of date'
stage = 'resolved'
status = 'closed'
superseder = '22578'
type = 'enhancement'
url = 'https://bugs.python.org/issue16007'
versions = ['Python 3.4']

[lingster]

Hi,

I would like to make a small improvement to the exception raised when _compile() function fails with exception on line 245.

Please see attached amended file. The line of code is amended from:

raise error, v # invalid expression

to:

raise error, r'%s with pattern: %s' % (v, pattern) # invalid expression

As this is my first contribution, please let me know if this is the correct procedure to submit changes.

Kind regards,
Ling

[benjaminp]

Please provide a patch in unified diff format.

[RamchandraApte]

As this is my first contribution, please let me know if this is the correct procedure to submit changes.

This is the correct procedure. http://docs.python.org/devguide/ (the Python Devlopment Guide) is a guide on how to contribute to Python.
Hope you enjoy your Python development experience!

[WaelAlJishi]

The attached file is very different to the current source, including the docstring. Is this from python 2.x?

[BreamoreBoy]

The original try/except that leads to the raise does not exist in the latest 3.5 code. I'd be inclined to close this as "out of date" for compatibility reasons if nothing else.

[serhiy-storchaka]

Since 3.5 (bpo-22578) the re.error exception has attributes that contain a pattern and the position of the error. Error message contains the position of the error. A pattern is not included in the error message for purpose: it can be too long. Usually the pattern is clear from the context, and you always can check the pattern attribute.