NOTE: The current preferred location for bug reports is the GitHub issue tracker.
Bug 496 - remove code.elem.strict and put code.elem into common.elem.phrasing instead
remove code.elem.strict and put code.elem into common.elem.phrasing instead
Status: RESOLVED FIXED
Product: Validator.nu
Classification: Unclassified
Component: HTML5 schema
HEAD
All All
: P2 normal
Assigned To: Michael[tm] Smith
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-04-30 06:25 CEST by Michael[tm] Smith
Modified: 2009-05-01 04:41 CEST (History)
1 user (show)

See Also:


Attachments
patch with proposed change (622 bytes, patch)
2009-04-30 07:29 CEST, Michael[tm] Smith
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael[tm] Smith 2009-04-30 06:25:40 CEST
The syntax/relaxng/phrase.rnc file currently defines both a code.elem pattern and a code.elem.strict, with corresponding code.inner and code.inner.strict patterns for their content models. But the definitions for code.inner and code.inner.strict are actually identical (=common.inner.phrasing). And the attribute model for code.elem and code.elem.strict is also identical (=code.attrs). So code.elem and code.elem.strict are actually identical.

code.elem is actually not currently reference by any other pattern, code.elem.strict is in common.elem.phrasing. So to rationalize things, common.elem.phrasing should reference code.elem instead, and the code.*.strict stuff can just be removed.

(Based on doing some quick archaeology through the revision history, I can see that it seems like the code.*.strict stuff is the last remaining bit of cruft left over from an earlier (and more complicated) incarnation of the schema, in which the *.*.strict stuff was used in many other content models.)
Comment 1 Michael[tm] Smith 2009-04-30 07:29:05 CEST
Created attachment 94 [details]
patch with proposed change
Comment 2 Henri Sivonen 2009-05-01 03:11:08 CEST
Looks good.
Comment 3 Michael[tm] Smith 2009-05-01 04:41:17 CEST
syntax r416 - http://svn8.cvsdude.com/vvc/whattf/syntax?view=revision&revision=416