On 03/ 1/11 01:17 PM, Adam Jackson wrote: > This joke is no longer funny. > > Signed-off-by: Adam Jackson <a...@redhat.com> > --- > glx/glapi.c | 22 ++-------------------- > 1 files changed, 2 insertions(+), 20 deletions(-) > > diff --git a/glx/glapi.c b/glx/glapi.c > index d6a568e..fd4eb92 100644 > --- a/glx/glapi.c > +++ b/glx/glapi.c > @@ -226,24 +226,6 @@ PUBLIC void *_glapi_Context = NULL; > #endif /* defined(GLX_USE_TLS) */ > /*@}*/ > > - > -/** > - * strdup() is actually not a standard ANSI C or POSIX routine. > - * Irix will not define it if ANSI mode is in effect. > - */ > -static char * > -str_dup(const char *str) > -{ > - char *copy; > - copy = (char*) malloc(strlen(str) + 1); > - if (!copy) > - return NULL; > - strcpy(copy, str); > - return copy; > -} > - > - > - > /** > * We should call this periodically from a function such as glXMakeCurrent > * in order to test if multiple threads are being used. > @@ -713,7 +695,7 @@ add_function_name( const char * funcName ) > if (entrypoint != NULL) { > entry = & ExtEntryTable[NumExtEntryPoints]; > > - ExtEntryTable[NumExtEntryPoints].name = str_dup(funcName); > + ExtEntryTable[NumExtEntryPoints].name = strdup(funcName); > ExtEntryTable[NumExtEntryPoints].parameter_signature = NULL; > ExtEntryTable[NumExtEntryPoints].dispatch_offset = ~0; > ExtEntryTable[NumExtEntryPoints].dispatch_stub = entrypoint; > @@ -858,7 +840,7 @@ _glapi_add_dispatch( const char * const * function_names, > } > } > > - entry[i]->parameter_signature = str_dup(real_sig); > + entry[i]->parameter_signature = strdup(real_sig); > fill_in_entrypoint_offset(entry[i]->dispatch_stub, offset); > entry[i]->dispatch_offset = offset; > }
Reviewed-by: Alan Coopersmith <alan.coopersm...@oracle.com> -- -Alan Coopersmith- alan.coopersm...@oracle.com Oracle Solaris Platform Engineering: X Window System _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel