--- a/model/comments/issue-438-james.txt	Sun Jul 08 21:49:27 2012 +0100
+++ b/model/comments/issue-438-james.txt	Sun Jul 08 21:59:18 2012 +0100
@@ -71,6 +71,15 @@
    > 38-42 ambiguous.  On the other hand, lots of URIs use '-'.  Suggested
    > fix: Allow '-' only if it is not the first or last character (could be
    > handled similar to '.').
+
+'-' can be parsed as PN_CHARS
+but not as PN_CHARS_U nor PN_CHARS_OTHERS.
+
+So, '-' is not allow a valid PN_LOCAL
+
+Note '~'  is valid PN_CHARS_OTHERS, maybe it's the origin of the confusion?
+
+
    > 
    > 14.  S4.  Suggest pointing out that bundles cannot be nested (because
    > bundles aren't expressions)

--- a/model/comments/issue-438-khalid.txt	Sun Jul 08 21:49:27 2012 +0100
+++ b/model/comments/issue-438-khalid.txt	Sun Jul 08 21:59:18 2012 +0100
@@ -16,15 +16,29 @@
    >   respectively."
    > 
    > - In Section 2.5: "cooments" -> "comments"
+
+fixed
+
    > 
    > - In Section, 3.1.2, I suspect that "time" is non terminal, and
    >   therefore, the reader may expect it to be broken down into terminals
    >   terms. I know this is beyond the scope of PROV-DM, but it would be
    >   good to add a sentence saying that.
+
+Time is defined in section 3.7.3.2 as the DATETIME terminal, and the
+definition can be found in xml schema.
+
+[49]   	time	   ::=   	DATETIME
+
    > 
    > - There are two terms that are used to define identfiers: "Identifier"
    >   and "OptionalIdentifier". Why not use "Identifier" and when it is
    >   optional use "(Identifier)?" as specified in Section 2.2?
+
+It's production [10], which also include ";"
+
+[10]   	optionalIdentifier	   ::=   	( identifierOrMarker ";" )?
+
    > 
    > - In section 3.1.4, it is said that "Even though the production
    >   usageExpression allows for expressions used(a2, -, -) and used(-;