On Wed, Feb 08, 2012 at 06:35:10PM -0800, Chase Douglas wrote:
> Signed-off-by: Chase Douglas <[email protected]>
> ---
> src/synaptics.c | 12 ++++++++++++
> src/synproto.h | 6 ++++++
> 2 files changed, 18 insertions(+), 0 deletions(-)
>
> diff --git a/src/synaptics.c b/src/synaptics.c
> index a4c1e5a..48fa519 100644
> --- a/src/synaptics.c
> +++ b/src/synaptics.c
> @@ -2809,3 +2809,15 @@ CalculateScalingCoeffs(SynapticsPrivate *priv)
> priv->vert_coeff = 1;
> }
> }
> +
> +struct SynapticsHwState *
> +SynapticsHwStateAlloc(SynapticsPrivate *priv)
> +{
> + return calloc(1, sizeof(struct SynapticsHwState));
> +}
> +
> +void
> +SynapticsHwStateFree(struct SynapticsHwState *hw)
can we do the double-pointer thing here? I find it reduces bugs if the
FreeFoobar() function also resets the pointers to NULL.
Cheers,
Peter
> +{
> + free(hw);
> +}
> diff --git a/src/synproto.h b/src/synproto.h
> index 6962bbf..7ec5142 100644
> --- a/src/synproto.h
> +++ b/src/synproto.h
> @@ -32,6 +32,9 @@
> #include <xf86Xinput.h>
> #include <xisb.h>
>
> +struct _SynapticsPrivateRec;
> +typedef struct _SynapticsPrivateRec SynapticsPrivate;
> +
> /*
> * A structure to describe the state of the touchpad hardware (buttons and
> pad)
> */
> @@ -90,4 +93,7 @@ extern struct SynapticsProtocolOperations
> event_proto_operations;
> extern struct SynapticsProtocolOperations psm_proto_operations;
> #endif /* BUILD_PSMCOMM */
>
> +extern struct SynapticsHwState *SynapticsHwStateAlloc(SynapticsPrivate
> *priv);
> +extern void SynapticsHwStateFree(struct SynapticsHwState *hw);
> +
> #endif /* _SYNPROTO_H_ */
> --
> 1.7.8.3
>
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
