[osis-core] Regex question

[Patrick Durusau]

Guys,

Working on the new regex and noticed that we currently have for grain:

(@(cp:\[(\p{Nd})*\]|s:\[(\p{L}|\p{N}|\s)*\]))?

Note that the "*" operator means zero or more.

IF someone put the grain syntax in (yes, need to ditch the ":" as well, 
but it is now in 1.7) shouldn't we require some content in between the
[ ]'s?

Easy enough to fix using the "+" operator, which means one or more.

Would give us cheap error detection on grains that have no content.

Noticed it when I was writing the addition to the regex for Todd's 
occurrrence indicator.

Hope everyone is having a great day!

Patrick

-- 
Patrick Durusau
Director of Research and Development
Society of Biblical Literature
Patrick.Durusau@sbl-site.org
Chair, V1 - Text Processing: Office and Publishing Systems Interface
Co-Editor, ISO 13250, Topic Maps -- Reference Model

Topic Maps: Human, not artificial, intelligence at work!