[Xcb] [PATCH proto] Remove valueparam support from xcbgen parsers and schema definition

[Jaya Tiwari]

Removed all the valueparam occurences from parsers and xml schema as
well along with all the protocol
definitions as valueparam has been replaced by switch bit-case

Signed-off-by: Jaya Tiwari <tiwari.jaya18 at gmail.com>
---
 src/xcb.xsd      | 11 -----------
 xcbgen/expr.py   |  7 -------
 xcbgen/xtypes.py |  7 -------
 3 files changed, 25 deletions(-)

diff --git a/src/xcb.xsd b/src/xcb.xsd
index 85f5bc2..6c70cdd 100644
--- a/src/xcb.xsd
+++ b/src/xcb.xsd
@@ -182,15 +182,6 @@ authorization from the authors.
     </xsd:complexType>
   </xsd:element>

-  <!-- BITMASK/LISTofVALUE parameter pairs. -->
-  <xsd:element name="valueparam">
-    <xsd:complexType>
-      <xsd:attribute name="value-mask-type" type="xsd:string" use="required" />
-      <xsd:attribute name="value-mask-name" type="xsd:string" use="required" />
-      <xsd:attribute name="value-list-name" type="xsd:string" use="required" />
-    </xsd:complexType>
-  </xsd:element>
-
   <xsd:group name="fields">
     <xsd:choice>
       <xsd:element ref="pad" />
@@ -296,7 +287,6 @@ authorization from the authors.
             <xsd:choice minOccurs="0" maxOccurs="unbounded">
               <xsd:group ref="fields" />
               <xsd:element ref="exprfield" />
-              <xsd:element ref="valueparam" />
             </xsd:choice>
             <xsd:choice minOccurs="0" maxOccurs="1">
               <xsd:element ref="switch" />
@@ -306,7 +296,6 @@ authorization from the authors.
                 <xsd:sequence>
                   <xsd:choice minOccurs="1" maxOccurs="unbounded">
                     <xsd:group ref="fields" />
-                    <xsd:element ref="valueparam" />
                   </xsd:choice>
                   <xsd:choice minOccurs="0" maxOccurs="1">
                     <xsd:element ref="switch" />
diff --git a/xcbgen/expr.py b/xcbgen/expr.py
index e4fb06e..6b70204 100644
--- a/xcbgen/expr.py
+++ b/xcbgen/expr.py
@@ -61,13 +61,6 @@ class Expression(object):
             # Standard list with a fieldref
             self.lenfield_name = elt.text

-        elif elt.tag == 'valueparam':
-            # Value-mask.  The length bitmask is described by attributes.
-            self.lenfield_name = elt.get('value-mask-name')
-            self.lenfield_type = elt.get('value-mask-type')
-            self.lenwire = True
-            self.bitfield = True
-
         elif elt.tag == 'op':
             # Op field.  Need to recurse.
             self.op = elt.get('op')
diff --git a/xcbgen/xtypes.py b/xcbgen/xtypes.py
index 45d7568..b4cbce0 100644
--- a/xcbgen/xtypes.py
+++ b/xcbgen/xtypes.py
@@ -184,8 +184,6 @@ class ListType(Type):
         if elt.tag == 'list':
             elts = list(elt)
             self.expr = Expression(elts[0] if len(elts) else elt, self)
-        elif elt.tag == 'valueparam':
-            self.expr = Expression(elt, self)

         self.size = member.size if member.fixed_size() else None
         self.nmemb = self.expr.nmemb if self.expr.fixed_size() else None
@@ -328,11 +326,6 @@ class ComplexType(Type):
                 fkey = child.get('type')
                 type = ListType(child, module.get_type(fkey),
*self.lenfield_parent)
                 visible = True
-            elif child.tag == 'valueparam':
-                field_name = child.get('value-list-name')
-                fkey = 'CARD32'
-                type = ListType(child, module.get_type(fkey),
*self.lenfield_parent)
-                visible = True
             elif child.tag == 'switch':
                 field_name = child.get('name')
                 # construct the switch type name from the parent type
and the field name


-- 
Regards,
Jaya