On 11/08/2011 07:08 PM, Peter Hutterer wrote: > On Mon, Nov 07, 2011 at 06:01:32PM -0800, Chase Douglas wrote: >> On 11/07/2011 01:39 PM, Peter Hutterer wrote: >>> Currently not needed since the InputClientRec is a self-contained struct. As >>> part of the touch rework that won't be the case in the future and a function >>> to allocate/free memory appropriately is required. >>> >>> Signed-off-by: Peter Hutterer <[email protected]> >>> --- >>> Xi/exevents.c | 23 ++++++++++++++++++----- >>> 1 files changed, 18 insertions(+), 5 deletions(-) >>> >>> diff --git a/Xi/exevents.c b/Xi/exevents.c >>> index 53db036..7f8e6db 100644 >>> --- a/Xi/exevents.c >>> +++ b/Xi/exevents.c >>> @@ -1628,6 +1628,19 @@ SelectForWindow(DeviceIntPtr dev, WindowPtr pWin, >>> ClientPtr client, >>> return Success; >>> } >>> >>> +static void >>> +FreeInputClient(InputClientsPtr *other) >>> +{ >>> + free((*other)); >> >> I thought double parens has been taboo for quite some time now... why >> did you use one here? > > copy/paste error, amended locally. thanks.
Reviewed-by: Chase Douglas <[email protected]> _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
