Apparently CMake on Windows is case insensitive and the library and
the executable cannot have the same name. Use a different name for
the executable on Windows.
The MS Visual Studio preprocessor somehow translates OUT in a blank
space so OUT cannot be used as name of an ofstream. Replace it with
OutputFile
Add Minuit and Gui libraries to the list of ROOT libraries to be
linked (addition of Minuit already done for Linux/MacOS, Gui is
a default there)
stdint.h is a C99 extension which is not compatible with using CINT
on Windows. Make sure that stdint.h is not included for the ROOTCINT
step on Windows and insert the appropriate type definitions instead
Make sure that all the classes have the DLLEXPORT preprocessor
directive if they have to be included in a shared library.:
Remove the include api.h statement (inherited from PixTest.hh)
and remove the semicolon from the ClassDef lines (ClassDef is
a preprocessor directive)
Remove using namespace std from header files
Remove using namespace pxar from header files
Use new include file pxardllexport for the definition of DLLEXPORT
macro and avoiding duplicating it.
Use new include file pxardllexport for the definition of the DLLEXPORT
macro and avoiding duplicating it
Include file for the DLLEXPORT macro, ensuring that the definitions
are done in a way that is compatible with the OS used and with running
ROOTCINT
Remove include api.h (it comes through include PixTest.h), remove
the semicolon after ClassDef, include the DLLEXPORT preprocessor
directive in each class
Move the include constants.h to the source file, remove it from the header
file
Add the DLLEXPORT directive in each class and make sure that either
through include PixGui.h (that includes include api.h) or through
an explicit include pxardllexport.h this preprocessor macro is available
Remove the tupedef char __signed statement that is not needed.
Remove a Windows compilation warning (only 1 constructor needed,
the default value for hubId makes the other constructor redundant)
Remove a compilation warning on Windows (casting a uint16_t as bool,
now bool value explicitly obtained with a test)
Remove a compilation warning on Linux (casting a uint16_t as bool,
now a boolean is obtained from an actual test)
Remove a warning on Windows
Add core/util to the list of include directories (for pxardllexport.h)Add
core/util to the list of include directories (for pxardllexport.h)Add
core/util to the list of include directories (for pxardllexport.h)Add
core/util to the list of include directories (for pxardllexport.h)Add
core/util to the list of include directories (for pxardllexport.h)Add
core/util to the list of include directories (for pxardllexport.h)Add
core/util to the list of include directories (for pxardllexport.h)Add
core/util to the list of include directories (for pxardllexport.h)Add
core/util to the list of include directories (for pxardllexport.h)