Ethereal-dev: RE : RE : RE : [Ethereal-dev] enabling libethereal.dll with MSVC (update)
Yes, I was thinking about a header file. Indeed, there are many files to
modify but when done, the management is easy.
-----Original Message-----
From: Lars Roland [mailto:lars.roland@xxxxxxx]
Sent: Monday, May 03, 2004 7:52 PM
To: RABRET Laurent FTRD/DAC/ISS
Cc: Ethereal development
Subject: Re: RE : RE : [Ethereal-dev] enabling libethereal.dll with MSVC
(update)
So is this single file a header file?
If so, I would have to include it everywhere, where we have at least one
function to export, right? And I would have to remove the declaration of
those functions from other header files to avoid duplicates ?
RABRET Laurent FTRD/DAC/ISS wrote:
> Wrong, if you create a single file dedicated to dll import/export
> function declaration, the complexity is the same compared to maintain
a
> def file.
> I do agree, using __declspec(dllexport) requires to add additional
> pre-processor directive macro.
> Regards
> Laurent
>
> -----Original Message-----
> From: ethereal-dev-bounces@xxxxxxxxxxxx
> [mailto:ethereal-dev-bounces@xxxxxxxxxxxx] On Behalf Of Lars Roland
> Sent: Monday, May 03, 2004 6:58 PM
> To: Ethereal development
> Subject: Re: RE : [Ethereal-dev] enabling libethereal.dll with MSVC
> (update)
>
> Yes but I think it a lot easier to create, maintain and review a def
> file with 478 lines (maainly copy'n'pasting) than finding and changing
> 487 defintions distributed throughout the code. :)
> It might be problematic to use __declspec(dllexport) as long as we
still
>
> allow this code to be linked statically into ethereal depending just
on
> an option in config.nmake. You will probably need some additional
> preprocessor directives and macros.
>
> I do not want to complicate the first step towards a stable and tested
> libethereal.dll. Can my patches checked in so it gets tested?
> The option in config.nmake to build ethereal with a libethereal.dll is
> off by default so it should not break anyone's build.
>
> However, we might think about replacing the def file for wiretap
first,
> to see if this is possible.
>
>
> RABRET Laurent FTRD/DAC/ISS wrote:
>
>
>>Yes and "def" file usage is no longer recommended (since a few years).
>>Using __declspec(dllimport) and __declspec(dllexport) also slightly
>>improve code efficiency (suppress one indirection). You must note it
>
> is
>
>>possible to add export functions in the dll without rebuilding the
>>executable.
>>Regards
>>Laurent
>>
>>-----Original Message-----
>>From: ethereal-dev-bounces@xxxxxxxxxxxx
>>[mailto:ethereal-dev-bounces@xxxxxxxxxxxx] On Behalf Of Guy Harris
>>Sent: Monday, May 03, 2004 11:02 AM
>>To: Ethereal development
>>Subject: Re: [Ethereal-dev] enabling libethereal.dll with MSVC
>
> (update)
>
>>Speaking of __declspec(), would using __declspec(dllexport) allow us
>
> to
>
>>get rid of libethereal.def?
>>
>>_______________________________________________
>>Ethereal-dev mailing list
>>Ethereal-dev@xxxxxxxxxxxx
>>http://www.ethereal.com/mailman/listinfo/ethereal-dev
>>
>>_______________________________________________
>>Ethereal-dev mailing list
>>Ethereal-dev@xxxxxxxxxxxx
>>http://www.ethereal.com/mailman/listinfo/ethereal-dev
>>
>
>
> _______________________________________________
> Ethereal-dev mailing list
> Ethereal-dev@xxxxxxxxxxxx
> http://www.ethereal.com/mailman/listinfo/ethereal-dev
>
