mistakes in XSD in transitionCondition and joinCondition syntax description

Hi, all editors,

When Yaron and I were cooking the formal proposal for Issue 103, we
found a syntax mistake about joinCondition in Section 6.2: 

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

<targets>?            

   <target linkName="ncname">+

      <joinCondition expressionLanguage="anyURI"?>?

        ... bool-expr ... 

      </joinCondition>

   </target>

</targets>

<sources>?

   <source linkName="ncname">+

      <transitionCondition expressionLanguage="anyURI"?>?

        ... bool-expr ... 

      </transitionCondition>

   </source>

</sources>

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

It is not in sync with the rest of the spec. 

I guess it should become: 

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

<targets>?            

   <joinCondition expressionLanguage="anyURI"?>?

        ... bool-expr ... 

   </joinCondition>

   <target linkName="ncname"/>+

</targets>

<sources>?

   <source linkName="ncname">+

      <transitionCondition expressionLanguage="anyURI"?>?

        ... bool-expr ... 

      </transitionCondition>

   </source>

</sources>

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

Also, in the XSD, I believe the "minOccurs" is missing in XSD for
joinCondition and transitionCondition:

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

    <complexType name="tTargets">

        <complexContent>

            <extension base="bpws:tExtensibleElements">

                <sequence>

                    <element name="joinCondition"
type="bpws:tCondition" minOccurs="0"/>

                    <element name="target" type="bpws:tTarget"
maxOccurs="unbounded"/>

                </sequence>

            </extension>

        </complexContent>

    </complexType>

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

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

    <complexType name="tSource">

        <complexContent>

            <extension base="bpws:tExtensibleElements">

                <sequence>

                    <element name="transitionCondition"
type="bpws:tCondition" minOccurs="0"/>

                </sequence>

                <attribute name="linkName" type="NCName"
use="required"/>

            </extension>

        </complexContent>

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

Is this change big enough to file a bug issue ?

Thanks!

Regards,

Alex Yiu