‘_’ was not requested for namespaces.

From: Michael Kalygin [mailto:notifications@github.com]
Sent: Friday, October 25, 2013 6:51 PM
To: alterm4nn/ChronoZoom
Cc: Sergey Berezin
Subject: Re: [ChronoZoom] Triples update: more default prefixes, '_' support, incoming triples (#1200)


In Source/Chronozoom.Entities/TripleName.cs:

> @@ -36,9 +38,12 @@ public string Prefix

>              get { return prefix; }

>          }

>

> -        private static readonly Regex _prefixReg = new Regex(@"^([a-z0-9]+):([a-z0-9\-]*)$", RegexOptions.Compiled | RegexOptions.IgnoreCase);

> +        private static readonly Regex _prefixReg = new Regex(@"^([a-z0-9_]+):([a-z0-9_\-]*)$", RegexOptions.Compiled | RegexOptions.IgnoreCase);

>          private static readonly Regex _namespaceReg = new Regex(@"^http://(?:www.)*chronozoom.com/([a-z0-9]+)#([a-z0-9\-]*)$", RegexOptions.Compiled | RegexOptions.IgnoreCase);

Shouldn't this Regex contain "_" also?

—
Reply to this email directly or view it on GitHub<https://github.com/alterm4nn/ChronoZoom/pull/1200/files#r7218097>.