Annotation namespace is lexed as whitespace

[ShortDevelopment]

Thanks for the quick release!

Sadly I discovered one more bug with the annotation lexing 😅.
(Sry for all these weird ede-cases 🙈)

POC

Consider the following OpenQASM 3 snippet:

@annotation
@annotation value
@annotation.long.namespace
@annotation.long.namespace  value
 @annotation.long.namespace
 @annotation.long.namespace  value

Issue

Using the current regex, the last namespace segment get's captured in a separate capture-group that will currently be marked as whitespace instead of the actual whitespace (last capture-group) at the end.

openqasm-pygments/src/openqasm_pygments/qasm3.py

Lines 83 to 84 in e7eed12

	r"^([ \t]*)(@\w+(\.\w+)*)([ \t]*)",
	bygroups(token.Whitespace, token.Name.Decorator, token.Whitespace),

https://regex101.com/r/a6mCWI/1

Solution

Use a non-capturing group for the namespace part.

- r"^([ \t]*)(@\w+(\.\w+)*)([ \t]*)",
+ r"^([ \t]*)(@\w+(?:\.\w+)*)([ \t]*)",

https://regex101.com/r/DsSRKZ/1

[jakelishman]

Ah nuts, I don't know how I missed that in the tests - I even thought "these nested groups might messing up my ordering". Would you be able to make the fixing PR?

[ShortDevelopment]

Would you be able to make the fixing PR?

Sure 🙂