This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 1715 - [FS] editorial: 5.11 Module Import
Summary: [FS] editorial: 5.11 Module Import
Status: ASSIGNED
Alias: None
Product: XPath / XQuery / XSLT
Classification: Unclassified
Component: Formal Semantics 1.0 (show other bugs)
Version: Last Call drafts
Hardware: All All
: P2 minor
Target Milestone: ---
Assignee: Michael Dyck
QA Contact: Mailing list for public feedback on specs from XSL and XML Query WGs
URL:
Whiteboard: need to create erratum and edit 1.1 d...
Keywords:
: 1900 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-07-18 19:56 UTC by Michael Dyck
Modified: 2007-09-21 01:06 UTC (History)
1 user (show)

See Also:


Attachments

Description Michael Dyck 2005-07-18 19:56:42 UTC
5.11 Module Import

SCP / rule 1 / premise (2|3)
"statEnv1 + varType(fs:local-variables(...))"
"statEnv2 + localFunc(fs:local-functions(...))"
    This syntax is unspecified.

SCP / rule 1 / premise 3
"statEnv2 + localFunc(...)"
    s/localFunc/funcType/

SCP / rule 1 / conclusion
"import module (namespace NCName =)? ..."
    If NCName appears, it should be bound to the target namespace in
    statEnv.namespace.

Notation 2
"The rules below depend on the following auxilliary judgments."
    Presumably, this should be followed by declarations of the
    "=>import_variables" and "=>import_functions" judgment forms.

    Such a Notation section should precede the DCP section though.

"This judgment adds each ..." (twice)
    s/judgment/rule/

DCP / rule 2
dynEnv2 ; URI |- (expanded-QName2(Type2,1, ..., Type2,n)), ยทยทยท,"
                 (expanded-QNamek(Typek,1, ..., Typek,n)) => ...
    This requires that all functions imported from a module have the same
    number of arguments (n), which is not what you want.

    You could fix this with subscripts-on-subscripts, but that would make
    the rule harder to read and understand. The better route, which
    actually makes the rule *easier* to read and understand, is to
    introduce a Formal symbol for (this kind of) function signature.

"to the input dynamic context."
    "input"? Maybe s/input/importing/

DCP / rule 3 / premise (2|3)
"fs:local-foo(statEnv2, URILiteral1)"
    Delete the subscript 1.

"URILiteral ; dynEnvi |-"
    Change to "dynEnvi ; URILiteral |-" to match previous rules.

dynEnv2, statEnv2
    It's not clear why dynEnv2 isn't used, or where statEnv2 comes from.
Comment 1 Per Bothner 2005-08-29 03:11:51 UTC
*** Bug 1900 has been marked as a duplicate of this bug. ***
Comment 2 Jerome Simeon 2006-04-17 04:58:46 UTC
Fixed as suggested when possible, since some of those comments were taken over by events.
- Jerome
Comment 3 Michael Dyck 2006-09-08 02:06:33 UTC
The 2006-06 CR does not have fixes for some of the items, specifically:

Notation 3
"The rules below depend on the following auxilliary judgments."
    But no auxiliary judgments are set out.

Notation 3 / rule (2|4)
    This requires that all functions imported from a module have the same
    number of arguments (n), which is not what you want.
    (This comment dates back to April 2004.)

Notation 4 / rule 2
    It's not clear why dynEnv2 isn't used, or where statEnv2 comes from.

    (And by the way, premises 1 and 2 are identical.)
Comment 4 Michael Dyck 2007-09-21 01:06:35 UTC
The second item of Comment #3 has been resolved by the fix for FS erratum E006.