win: work around __pfnDliNotifyHook2 type change

Visual Studio 2015 Update 3 defines __pfnDliNotifyHook2 as const. The decltype specifier makes the declaration work across all supported versions of VS. It also requires that the source be compiled as C++. Fixes: #949 PR-URL: #952 Reviewed-By: bnoordhuis - Ben Noordhuis <[email protected]>
nodejs · Jun 13, 2016 · f31482e · f31482e · Fishrock123 · Jun 13, 2016
1 parent 3df8222
commit f31482e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/addon.gypi b/addon.gypi
@@ -43,7 +43,7 @@
         'conditions': [
           [ 'OS=="win"', {
             'sources': [
-              '<(node_gyp_dir)/src/win_delay_load_hook.c',
+              '<(node_gyp_dir)/src/win_delay_load_hook.cc',
             ],
             'msvs_settings': {
               'VCLinkerTool': {

diff --git a/src/win_delay_load_hook.c → src/win_delay_load_hook.cc b/src/win_delay_load_hook.c → src/win_delay_load_hook.cc
@@ -31,6 +31,6 @@ static FARPROC WINAPI load_exe_hook(unsigned int event, DelayLoadInfo* info) {
   return (FARPROC) m;
 }
 
-PfnDliHook __pfnDliNotifyHook2 = load_exe_hook;
+decltype(__pfnDliNotifyHook2) __pfnDliNotifyHook2 = load_exe_hook;
 
 #endif