Reviewed-by: Jamey Sharp <[email protected]>

Of course, now there's only one caller of RRInitOutputPropertyValue;
is it worth inlining?

Jamey

On Sun, Jun 6, 2010 at 6:40 AM, Mikhail Gusarov <[email protected]> wrote:
> Reviewed-by: Mikhail Gusarov <[email protected]>
> ---
>  randr/rrproperty.c |   10 +---------
>  1 files changed, 1 insertions(+), 9 deletions(-)
>
> diff --git a/randr/rrproperty.c b/randr/rrproperty.c
> index 7b8e44e..c4dcb1a 100644
> --- a/randr/rrproperty.c
> +++ b/randr/rrproperty.c
> @@ -103,18 +103,10 @@ RRCreateOutputProperty (Atom property)
>  {
>     RRPropertyPtr   prop;
>
> -    prop = (RRPropertyPtr)malloc(sizeof(RRPropertyRec));
> +    prop = calloc(1, sizeof(RRPropertyRec));
>     if (!prop)
>        return NULL;
> -    prop->next = NULL;
>     prop->propertyName = property;
> -    prop->is_pending = FALSE;
> -    prop->range = FALSE;
> -    prop->immutable = FALSE;
> -    prop->num_valid = 0;
> -    prop->valid_values = NULL;
> -    RRInitOutputPropertyValue (&prop->current);
> -    RRInitOutputPropertyValue (&prop->pending);
>     return prop;
>  }
>
> --
> 1.7.1
>
> _______________________________________________
> [email protected]: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
>
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to