[nas] [patch] remove exported NoopDDA function

Jon Trulson jon at radscan.com
Wed Aug 30 18:48:05 MDT 2006


On Wed, 30 Aug 2006, Erik Auerswald wrote:

> Hi,
>
> On Wed, Aug 30, 2006 at 02:23:23PM +0200, Stefan Huehner wrote:
>> there is a empty NoopDDA function in dixutils.c which is exported and
>> listed in dix.h but only used in one place: resource.c . This patch
>> moves this functions to resource.c and modifies the parameters/return
>> value to match the expected values there.
>
> The DeleteFuncs seem to be of type (int *) (pointer, AuID), not
> (int *) (void).
>

         Erik is correct...

         I did not apply this patch anyways, as it really *is* useful
         to have a no-op callback in dix (ahem, sorry: dia :).  Athough
         it is only being used internally in one place at the moment,
         it will be useful to have if extensions or other server
         modifications are ever made that create and manipulate new
         server resources - so I think we should leave it alone...

         I did, however, go though and fix the appropriate typedef, and
         correct the various build issues that entailed :)

-- 
Jon Trulson
mailto:jon at radscan.com http://radscan.com/~jon
#include <std/disclaimer.h>
"No Kill I" -Horta




More information about the Nas mailing list