Errors in Part 1 XSD schemas (except wml.xsd and sml.xsd) in the consolidated Part 1

[Rex Jaeschke]

See my responses in-line. Rex


> -----Original Message-----
> From: MURATA Makoto (FAMILY Given) [mailto:eb2m-mrt at asahi-net.or.jp]
> Sent: Saturday, March 05, 2011 2:49 AM
> To: e-sc34-wg4 at ecma-international.org
> Subject: Errors in Part 1 XSD schemas (except wml.xsd and sml.xsd) in the
> consolidated Part 1
> 
> I am using oXygen directory comparison to compare schemas in Appendix A
> and my schemas in the Assembla repository (trunk).
> 
> I have not covered wml.xsd or sml.xsd, which are too gigantic for oXygen
to
> do comparison.  But oXygen can hande the rest.
> 
> 1) dml-chart.xsd
> 
>     <xsd:restriction base="xsd:string">
> and <xsd:restriction> in ST_HPercentWithSymbol is missing.

==============================

My response:

I agree.

In DR 09-0033 of the DR log, the following xsd schema is shown:

<xsd:simpleType name="ST_HPercentWithSymbol">
	<xsd:restriction base="xsd:string">
		<xsd:pattern
value="0*(([5-9])|([1-9][0-9])|([1-4][0-9][0-9])|500)%"/>
	</xsd:restriction>
</xsd:simpleType>

which does include the lines that you report are missing from the
consolidated spec. And that's what WG4 agreed to.

However, if you look at the same schema in item 342 of AMD1, you see

<xsd:simpleType name="ST_HPercentWithSymbol">
  <xsd:pattern value="0*(([5-9])|([1-9][0-9])|([1-4][0-9][0-9])|500)%"/>
</xsd:simpleType>

where the lines you mention are missing. The error was made during the
creation of AMD1, and that was balloted without those lines, but with the
correct electronic schema. So now we have an electronic schema that is
correct, but an Annex A schema that is not. Given WG4's clear intent, I will
change Annex A to match the electronic schema.

==============================

> The default value 100 for @val in CT_HPercent should read 100%.

==============================

My response:

I agree; item 342 of AMD1 does include this correction; however, it appears
I missed making that change in the consolidation.

==============================

> The default value 100 for @val in CT_DepthPercent should read 100%.

==============================

My response:

I agree; item 343 of AMD1 does include this correction; however, it appears
I missed making that change in the consolidation.

==============================

> The def of ST_Perspective in dml-chart.xsd does  not use the COR1 to AMD1.

==============================

My response:

I don't understand. Item 274 of AMD1 changed ST_Perspective from being an
unsignedByte to a union of ST_PerspectivePercent. Item 2 of AMD1/COR1 said
that this change should be undone; that is, to set the type back to
unsignedByte. Item 4 of AMD1/COR1 said that the corresponding schema change
should be undone. So as far as I can tell, the consolidated text is correct.
If you disagree, please say what exactly is wrong.

==============================

> The default value 150 for @val in CT_GapAmount should be 150%  The default
> value 0 for @val inCT_Overlap" should be 0% The default value 100 for @val
> in CT_BubbleScale should be 100% The default value 75  for @val in
> CT_SecondPieSize should be 75% The default value 100 for @val in
> CT_LblOffset should be 100%

==============================

My response:

I agree; the corresponding items in AMD1 do include these corrections;
however, it appears I missed making those changes in the consolidation.

==============================

> 2) pml.xsd.
> 
> <xsd:attribute name="bwMode" type="a:ST_BlackWhiteMode"
> use="optional"/> should be in CT_GraphicalObjectFrame rather than
> CT_GraphicalObjectFrameNonVisual

==============================

My response:

I agree; I made the XSD change to the wrong type. (The corresponding RNG
change did go in the correct type, however).

==============================

> 3) shared-commonSimpleTypes.xsd
> <xs:union memberTypes="xsd:boolean"/>
> should be
> <xsd:union memberTypes="xsd:boolean"/>

==============================

My response:

I agree; however, note that while it was correct in the DR log, the error
was made during the creation of AMD1, and that was balloted without the
letter s. So now we have an electronic schema that is correct but an Annex A
schema that is not. Given WG4's clear intent, I will change Annex A to match
the electronic schema.

 ==============================

> 4) sml.xsd
> 
> ...main"http
> in sml.xsd should read
> ...main" xmlns:r="http...

==============================

My response:

I agree; the namespace information was inadvertently deleted when the item
320 of AMD1 was applied during consolidation.

 ==============================

> 
> Cheers,
> Makoto