----- Original Message -----
>
>
> ----- Original Message -----
> > Updated Branches:
> >   refs/heads/master a79a638d8 -> 53ebc2f72
> >
> >
> > TS-672 cleanup Win32 references
> >
> > nuke win32 completely.
> [snip]
> > http://git-wip-us.apache.org/repos/asf/trafficserver/blob/53ebc2f7/mgmt/api/include/mgmtapi.h
> > ----------------------------------------------------------------------
> > diff --git a/mgmt/api/include/mgmtapi.h
> > b/mgmt/api/include/mgmtapi.h
> > index bc4ca0b..8f51e97 100644
> > --- a/mgmt/api/include/mgmtapi.h
> > +++ b/mgmt/api/include/mgmtapi.h
> > @@ -39,21 +39,10 @@
> >   * System Specific Items
> >   
> > ***************************************************************************/
> >
> > -#if defined (_WIN32) && defined (_TS_EXPORT)
> > -#define tsapi __declspec( dllexport )
> > -#elif defined (_WIN32)
> > -#define tsapi __declspec( dllimport )
> > -#else
> >  #define tsapi
> > -#endif
> >
> > -#if defined (_WIN32)
> > -#define inkexp __declspec( dllexport )
> > -#define inkimp __declspec( dllimport )
> > -#else
> >  #define inkexp
> >  #define inkimp
> > -#endif
> >
> >  #if !defined(linux)
> >  #if defined (__SUNPRO_CC) || (defined (__GNUC__) || !
> >  defined(__cplusplus))
> >
>
> aside from making certain things more obvious, this macro only
> existed for the sake of WIN32. Now, I still think it makes sense
> to retain it, because it still makes these things more obvious,
> and also because one day we might want to try and port ATS to
> Windows again ;)

After your cleanup, this is what we're still left with:


-->8-------------

mgmt/web2/WebHttp.cc:203:    // access the .so/.dll plugin files.
example/add-header/add-header.c:29: *   (NT): AddHeader.dll "name1: value1" 
"name2: value2" ...
example/thread-pool/psi.c:29: *         (NT): psi.dll
example/bnull-transform/bnull-transform.c:30: *    (NT): BNullTransform.dll
example/append-transform/append-transform.c:32: *     (NT): AppendTransform.dll 
<filename>
example/redirect-1/redirect-1.c:30: *   (NT): Redirect.dll block_ip url_redirect
example/thread-1/thread-1.c:29: *       (NT): Thread.dll
example/null-transform/null-transform.c:30: *   (NT): NullTransform.dll
example/server-transform/server-transform.c:30: *       (NT): 
ServerTransform.dll
example/file-1/file-1.c:29: *   (NT): File.dll <filename1> <filename2> ...
proxy/http/TestUrl.cc:118:  
create_url("www.microsoft.com/isapi/redir.dll?TARGET=%2Foffice%2Fmigration%2F&nonie3home&homepage&&&&headline1&1006");
proxy/Plugin.cc:132:  handle = dll_open(path, (internal ? true : false));
proxy/Plugin.cc:134:    Error("unable to load '%s': %s", path, 
dll_error(handle));
proxy/Plugin.cc:138:  lic_req = (lic_req_func_t) dll_findsym(handle, 
"TSPluginLicenseRequired");
proxy/Plugin.cc:143:      dll_close(handle);
proxy/Plugin.cc:153:  init_func_w_handle_t inith = (init_func_w_handle_t) 
dll_findsym(handle, "TSPluginInitwDLLHandle");
proxy/Plugin.cc:159:  init = (init_func_t) dll_findsym(handle, "TSPluginInit");
proxy/Plugin.cc:161:    Error("unable to find TSPluginInit function '%s': %s", 
path, dll_error(handle));
proxy/Plugin.cc:162:    dll_close(handle);
proxy/Plugin.cc:170:  //dll_close(handle);


-----8<----------


I don't think the stuff in Plugin.cc or TestUrl.cc counts
and the examples are harmless.

i

--
Igor Galić

Tel: +43 (0) 664 886 22 883
Mail: i.ga...@brainsware.org
URL: http://brainsware.org/
GPG: 6880 4155 74BD FD7C B515  2EA5 4B1D 9E08 A097 C9AE

Reply via email to