I think we can drop WinDDK <= 5600 support entirely. If I'm not mistaken, this means supporting Visual Studio 2008 and later only, which is fine. Afaik the only "reason" to support Visual Studio 2005 and older is Win2k support, which we don't need. OpenVPN 2.1.3 was the last version with Win2k support anyways.
Besides that, it's an ACK. -- Samuli Seppänen Community Manager OpenVPN Technologies, Inc irc freenode net: mattock > Signed-off-by: Alon Bar-Lev <alon.bar...@gmail.com> > --- > src/prototypes.h | 2 +- > src/tapdrvr.c | 5 +---- > 2 files changed, 2 insertions(+), 5 deletions(-) > > diff --git a/src/prototypes.h b/src/prototypes.h > index 55454d5..18c566f 100755 > --- a/src/prototypes.h > +++ b/src/prototypes.h > @@ -202,7 +202,7 @@ VOID HookDispatchFunctions(); > > #if ENABLE_NONADMIN > > -#if DDKVER_MAJOR < 5600 > +#if defined(DDKVER_MAJOR) && DDKVER_MAJOR < 5600 > /* > * Better solution for use on Vista DDK, but possibly not compatible with > * earlier DDKs: > diff --git a/src/tapdrvr.c b/src/tapdrvr.c > index 0df076a..0f75b77 100755 > --- a/src/tapdrvr.c > +++ b/src/tapdrvr.c > @@ -36,9 +36,6 @@ > //====================================================== > > #include "common.h" > -#ifndef DDKVER_MAJOR > -#error DDKVER_MAJOR must be defined as the major number of the DDK Version > -#endif > > #define NDIS_MINIPORT_DRIVER > #define BINARY_COMPATIBLE 0 > @@ -66,7 +63,7 @@ > //======================================================== > #define ENABLE_NONADMIN 1 > > -#if DDKVER_MAJOR < 5600 > +#if defined(DDKVER_MAJOR) && DDKVER_MAJOR < 5600 > #include <ndis.h> > #include <ntstrsafe.h> > #include <ntddk.h>