vdpService_import.cpp not self contained

[ahtilego]

vdpService_import.cpp (that is in zip docs/vdpservice/versions/4.0/horizon-vdpservice-sdk-4.0.14304348675.zip) claims that it is self contained, and I would only have to add it to my project to use it. Unfortunately this is not true, it depends on a header helpers.h that is found in the samples/common subdirectory. And this header requires adding helpers.cpp from samples/common to my project, and this in turn requires my project to have stdafx.h.

An update that makes vdpService_import.cpp truly self-contained would be much appreciated.

[clayton1963]

Hi @ahtilego, vdpservice_import.cpp includes helpers.h because it requires code in helpers.cpp which is where the code that determines which version of Horizon is installed; Omnissa or VMware. That code was added in the latest SDK released when Horizon was rebranded from VMware to Omnissa. On the agent side it used to determine where vdpService.dll is located so that it can load it. On the client side it is used to know where to register your plugin. Since it is used on both sides it was placed in a common file to avoid code duplication. I agree that the comment is outdated and I will update it for the next SDK release. The source code to vdpservice_import.cpp was included so that partners are free to update it to suit their needs please feel free to do so in this case.