[x3d-public] SFImage regex

[Leonard Daly]

Andreas,

There is an error in the regex:
>
> Tangentially, repeated ranges also look not so nice but are apparently
> common: [0-9a-fA-F]
> [0-9]|[a-f|[A-F] reads better to me and that is why I preferred that.

There should be a ']' after 'f'. Personally, I prefer the first form 
[0-9a-fA-f] or even [a-fA-F\d]. If the entire string is case insensitive 
then use the language-specific 'i' mode or surround the entire string as 
in (?i)...[a-f\d]...(?-i) or shorter (?i)...[a-f\d]...

I prefer grouping all of the ranges together so that repeated 
vertical-ness is not present as it would be with ']|['. It also makes it 
clearer that the range of characters is all one unit.


-- 
*Leonard Daly*
3D Systems & Cloud Consultant
LA ACM SIGGRAPH Past Chair
President, Daly Realism - /Creating the Future/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://web3d.org/pipermail/x3d-public_web3d.org/attachments/20180611/9c8f6f22/attachment-0001.html>