[Xcb] [PATCH libxcb 09/18] c_client.py: remove end-of-function comments

Christian Linhart chris at DemoRecorder.com
Mon Oct 13 08:44:47 PDT 2014


These comments are hard to keep consistent. ( and some were already inconsistent )
So I think removing them is good.

Reviewed-by: Christian Linhart <chris at demorecorder.com>

On 10/12/14 20:58, Ran Benita wrote:
> Signed-off-by: Ran Benita <ran234 at gmail.com>
> ---
>  src/c_client.py | 16 ----------------
>  1 file changed, 16 deletions(-)
> 
> diff --git a/src/c_client.py b/src/c_client.py
> index 3310db1..0488ea5 100644
> --- a/src/c_client.py
> +++ b/src/c_client.py
> @@ -421,7 +421,6 @@ def _c_type_setup(self, name, postfix):
>                  if not module.namespace.is_ext or self.name[:2] == module.namespace.prefix:
>                      finished_sizeof.append(self.c_sizeof_name)
>                      _c_serialize('sizeof', self)
> -# _c_type_setup()
>  
>  def _c_helper_absolute_name(prefix, field=None):
>      """
> @@ -441,7 +440,6 @@ def _c_helper_absolute_name(prefix, field=None):
>      if field is not None:
>          prefix_str += _cpp(field.field_name)
>      return prefix_str
> -# _c_absolute_name
>  
>  def _c_helper_field_mapping(complex_type, prefix, flat=False):
>      """
> @@ -477,7 +475,6 @@ def _c_helper_field_mapping(complex_type, prefix, flat=False):
>                  all_fields.update(_c_helper_field_mapping(f.type, new_prefix, flat))
>  
>      return all_fields
> -# _c_field_mapping()
>  
>  def _c_helper_resolve_field_names (prefix):
>      """
> @@ -498,7 +495,6 @@ def _c_helper_resolve_field_names (prefix):
>          all_fields.update(_c_helper_field_mapping(obj, tmp_prefix, flat=True))
>  
>      return all_fields
> -# _c_helper_resolve_field_names
>  
>  def get_expr_fields(self):
>      """
> @@ -549,7 +545,6 @@ def get_expr_fields(self):
>  
>      resolved_fields = [all_fields[n][1] for n in resolved_fields_names]
>      return resolved_fields
> -# get_expr_fields()
>  
>  def resolve_expr_fields(complex_obj):
>      """
> @@ -572,7 +567,6 @@ def resolve_expr_fields(complex_obj):
>          if e not in all_fields and e not in unresolved:
>              unresolved.append(e)
>      return unresolved
> -# resolve_expr_fields()
>  
>  def get_serialize_params(context, self, buffer_var='_buffer', aux_var='_aux'):
>      """
> @@ -651,7 +645,6 @@ def get_serialize_params(context, self, buffer_var='_buffer', aux_var='_aux'):
>                  add_param(params, (p.c_field_const_type, '*', p.c_field_name))
>  
>      return (param_fields, wire_fields, params)
> -# get_serialize_params()
>  
>  def _c_serialize_helper_insert_padding(context, code_lines, space, postpone):
>      code_lines.append('%s    /* insert padding */' % space)
> @@ -676,7 +669,6 @@ def _c_serialize_helper_insert_padding(context, code_lines, space, postpone):
>  
>      # keep tracking of xcb_parts entries for serialize
>      return 1
> -# _c_serialize_helper_insert_padding()
>  
>  def _c_serialize_helper_switch(context, self, complex_name,
>                                 code_lines, temp_vars,
> @@ -729,7 +721,6 @@ def _c_serialize_helper_switch(context, self, complex_name,
>  #        code_lines.append('%s    xcb_buffer_len += xcb_block_len + xcb_pad;' % space)
>  
>      return count
> -# _c_serialize_helper_switch
>  
>  def _c_serialize_helper_switch_field(context, self, field, c_switch_variable, prefix):
>      """
> @@ -768,7 +759,6 @@ def _c_serialize_helper_switch_field(context, self, field, c_switch_variable, pr
>                                             c_field_names, prefix_str, field.c_field_name)
>  
>      return length
> -# _c_serialize_helper_switch_field()
>  
>  def _c_serialize_helper_list_field(context, self, field,
>                                     code_lines, temp_vars,
> @@ -835,7 +825,6 @@ def _c_serialize_helper_list_field(context, self, field,
>              code_lines.append('%s    xcb_block_len = xcb_parts[xcb_parts_idx].iov_len;' % space)
>  
>      return length
> -# _c_serialize_helper_list_field()
>  
>  def _c_serialize_helper_fields_fixed_size(context, self, field,
>                                            code_lines, temp_vars,
> @@ -901,7 +890,6 @@ def _c_serialize_helper_fields_fixed_size(context, self, field,
>                  length = '%d' % field.type.nmemb
>  
>      return (value, length)
> -# _c_serialize_helper_fields_fixed_size()
>  
>  def _c_serialize_helper_fields_variable_size(context, self, field,
>                                               code_lines, temp_vars,
> @@ -954,7 +942,6 @@ def _c_serialize_helper_fields_variable_size(context, self, field,
>          length = "%s(%s)" % (field.type.c_sizeof_name, var_field_name)
>  
>      return (value, length)
> -# _c_serialize_helper_fields_variable_size
>  
>  def _c_serialize_helper_fields(context, self,
>                                 code_lines, temp_vars,
> @@ -1045,7 +1032,6 @@ def _c_serialize_helper_fields(context, self,
>              need_padding = False
>  
>      return count
> -# _c_serialize_helper_fields()
>  
>  def _c_serialize_helper(context, complex_type,
>                          code_lines, temp_vars,
> @@ -1085,7 +1071,6 @@ def _c_serialize_helper(context, complex_type,
>      count += _c_serialize_helper_insert_padding(context, code_lines, space, False)
>  
>      return count
> -# _c_serialize_helper()
>  
>  def _c_serialize(context, self):
>      """
> @@ -1281,7 +1266,6 @@ def _c_serialize(context, self):
>      _c('')
>      _c('    return xcb_buffer_len;')
>      _c('}')
> -# _c_serialize()
>  
>  def _c_iterator_get_end(field, accum):
>      '''



More information about the Xcb mailing list