[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