Closed stijnvanbeek closed 3 years ago
Thanks for the PR. Because auto PR building is not installed for the public repo it's hard to verify the changes cross platform. For now I enabled auto PR building for the public repo, because the number of PRs are limited. Best to push a change and see if the system picks it up. I can't let anything pass without cross-platform verification.
[18:38:15] [Step 2/2] naputility.lib(threading.obj) : warning LNK4217: locally defined symbol ?enqueue@TaskQueue@nap@@QEAAXV?$function@$$A6AXXZ@std@@@Z (public: void __cdecl nap::TaskQueue::enqueue(class std::function<void __cdecl(void)>)) imported in function "public: virtual __cdecl nap::WorkerThread::~WorkerThread(void)" (??1WorkerThread@nap@@UEAA@XZ) [44040c56bd2f12b7\nap\packaging_build\modules\napaudio\mod_napaudio.vcxproj]
[18:38:15] [Step 2/2] naputility.lib(threading.obj) : warning LNK4217: locally defined symbol ??1TaskQueue@nap@@QEAA@XZ (public: __cdecl nap::TaskQueue::~TaskQueue(void)) imported in function "int `public: __cdecl nap::ThreadPool::ThreadPool(int,int,bool)'::`1'::dtor$4" (?dtor$4@?0???0ThreadPool@nap@@QEAA@HH_N@Z@4HA) [44040c56bd2f12b7\nap\packaging_build\modules\napaudio\mod_napaudio.vcxproj]
[18:38:15] [Step 2/2] naputility.lib(threading.obj) : error LNK2019: unresolved external symbol "__declspec(dllimport) const nap::WorkerThread::`vftable'" (__imp_??_7WorkerThread@nap@@6B@) referenced in function "public: __cdecl nap::WorkerThread::WorkerThread(bool,int)" (??0WorkerThread@nap@@QEAA@_NH@Z) [44040c56bd2f12b7\nap\packaging_build\modules\napaudio\mod_napaudio.vcxproj]
[18:38:15] [Step 2/2] 44040c56bd2f12b7\nap\packaging_bin\MSVC-x86_64-Release\mod_napaudio.dll : fatal error LNK1120: 1 unresolved externals [44040c56bd2f12b7\nap\packaging_build\modules\napaudio\mod_napaudio.vcxproj]
[18:32:32] [Step 2/2]44040c56bd2f12b7\nap\utility\src\utility\threading.cpp(14): warning C4273: 'nap::TaskQueue::TaskQueue': inconsistent dll linkage [44040c56bd2f12b7\nap\packaging_build\utility\naputility.vcxproj]
[18:32:32] [Step 2/2] 44040c56bd2f12b7\nap\utility\src\utility\threading.h(32): note: see previous definition of '{ctor}' (compiling source file 44040c56bd2f12b7\nap\utility\src\utility\threading.cpp)
Moving base of PR to public_beta. All PRs will be merged into public_beta until ready for release (tested).
These are a couple of changes to the NAP repository made by 4DSOUND:
these will concern Naivi most:
And these less so, as they are mainly audio changes: