Change CSP_PUBLIC to CSPIMPL_EXPORT for Windows build

Signed-off-by: Adam Glustein <adam.glustein@point72.com>

cpp/csp/adapters/kafka/KafkaAdapterManager.h

@@ -47,7 +47,7 @@ struct KafkaStatusMessageTypeTraits
 using KafkaStatusMessageType = csp::Enum<KafkaStatusMessageTypeTraits>;
 
 //Top level AdapterManager object for all kafka adapters in the engine
-class CSP_PUBLIC KafkaAdapterManager final : public csp::AdapterManager
+class CSPIMPL_EXPORT KafkaAdapterManager final : public csp::AdapterManager
 {
 public:
     KafkaAdapterManager( csp::Engine * engine, const Dictionary & properties );

cpp/csp/adapters/parquet/DialectGenericListReaderInterface.h

@@ -25,7 +25,7 @@ class DialectGenericListReaderInterface
 };
 
 template< typename T >
-class CSP_PUBLIC TypedDialectGenericListReaderInterface : public DialectGenericListReaderInterface
+class CSPIMPL_EXPORT TypedDialectGenericListReaderInterface : public DialectGenericListReaderInterface
 {
 public:
     using Ptr = std::shared_ptr<TypedDialectGenericListReaderInterface<T>>;

cpp/csp/adapters/parquet/ParquetInputAdapterManager.h

@@ -18,7 +18,7 @@ namespace csp::adapters::parquet
 
 
 //Top level AdapterManager object for all parquet adapters in the engine
-class CSP_PUBLIC ParquetInputAdapterManager final : public csp::AdapterManager
+class CSPIMPL_EXPORT ParquetInputAdapterManager final : public csp::AdapterManager
 {
 public:
     using GeneratorPtr = csp::Generator<std::string, csp::DateTime, csp::DateTime>::Ptr;

cpp/csp/adapters/parquet/ParquetOutputAdapterManager.h

@@ -21,7 +21,7 @@ class ParquetOutputFilenameAdapter;
 class ParquetDictBasketOutputWriter;
 
 //Top level AdapterManager object for all parquet adapters in the engine
-class CSP_PUBLIC ParquetOutputAdapterManager final : public csp::AdapterManager
+class CSPIMPL_EXPORT ParquetOutputAdapterManager final : public csp::AdapterManager
 {
 public:
     using FileVisitorCallback = std::function<void(const std::string &)>;

cpp/csp/adapters/websocket/ClientAdapterManager.h

@@ -40,7 +40,7 @@ struct WebsocketClientStatusTypeTraits
 
 using ClientStatusType = Enum<WebsocketClientStatusTypeTraits>;
 
-class CSP_PUBLIC ClientAdapterManager final : public AdapterManager
+class CSPIMPL_EXPORT ClientAdapterManager final : public AdapterManager
 {
 public:
     ClientAdapterManager(

cpp/csp/core/BasicAllocator.h

@@ -15,7 +15,7 @@ namespace csp
 {
 
 // Extremely basic non-thread safe fixed-size allocator
-class CSP_PUBLIC BasicAllocator
+class CSPIMPL_EXPORT BasicAllocator
 {
 public:
     //elemsize is size of a single alloc, blockSize is number of elements to 

cpp/csp/core/DynamicBitSet.h

@@ -9,7 +9,7 @@ namespace csp
 {
 
 template<typename NodeT = uint64_t, typename IndexT = int32_t>
-class CSP_PUBLIC DynamicBitSet
+class CSPIMPL_EXPORT DynamicBitSet
 {
 
 public:

cpp/csp/core/Enum.h

@@ -73,7 +73,7 @@ bool UnknownOnInvalidValue(long) { return false; }
 
 START_PACKED
 template<typename EnumTraits>
-struct CSP_PUBLIC Enum : public EnumTraits
+struct CSPIMPL_EXPORT Enum : public EnumTraits
 {
     using EnumV = typename EnumTraits::_enum;
     using Mapping = std::vector<std::string>;

cpp/csp/core/EnumBitSet.h

@@ -14,7 +14,7 @@ namespace csp
 //Utility class to hold enums as a bitmask ( where enum values are incremental from 0 )
 //enum must have a NUM_TYPES entry for number of entries
 template< typename EnumT >
-class CSP_PUBLIC EnumBitSet
+class CSPIMPL_EXPORT EnumBitSet
 {
     using value_type = uint64_t;
 

cpp/csp/core/Exception.h

@@ -10,7 +10,7 @@
 namespace csp
 {
 
-class CSP_PUBLIC Exception : public std::exception
+class CSPIMPL_EXPORT Exception : public std::exception
 {
 public:
     Exception( const char * exType, const std::string & description, const char * file, const char * func, int line ) :
@@ -59,7 +59,7 @@ class CSP_PUBLIC Exception : public std::exception
 };
 
 #define __FILENAME__ (strrchr(__FILE__, '/') ? strrchr(__FILE__, '/') + 1 : __FILE__)
-#define CSP_DECLARE_EXCEPTION( DerivedException, BaseException ) class CSP_PUBLIC DerivedException : public BaseException { public: DerivedException( const char * exType, const std::string &r, const char * file, const char * func, int line ) : BaseException( exType, r, file, func, line ) {} };
+#define CSP_DECLARE_EXCEPTION( DerivedException, BaseException ) class CSPIMPL_EXPORT DerivedException : public BaseException { public: DerivedException( const char * exType, const std::string &r, const char * file, const char * func, int line ) : BaseException( exType, r, file, func, line ) {} };
 
 CSP_DECLARE_EXCEPTION( AssertionError,     Exception )
 CSP_DECLARE_EXCEPTION( RuntimeException,   Exception )

cpp/csp/core/Generator.h

@@ -7,7 +7,7 @@
 namespace csp
 {
 template< typename V, typename ...Args >
-class CSP_PUBLIC Generator
+class CSPIMPL_EXPORT Generator
 {
 public:
     using Ptr = std::shared_ptr<Generator<V, Args...>>;

cpp/csp/core/Platform.h

@@ -15,7 +15,6 @@
 #undef GetMessage
 
 #define CSP_LOCAL
-#define CSP_PUBLIC __declspec(dllexport)
 
 #ifdef CSPTYPESIMPL_EXPORTS
 #define CSPTYPESIMPL_EXPORT __declspec(dllexport)
@@ -94,7 +93,6 @@ inline uint8_t ffs(uint64_t n)
 #define CSPTYPESIMPL_EXPORT __attribute__ ((visibility ("default")))
 
 #define CSP_LOCAL  __attribute__ ((visibility ("hidden")))
-#define CSP_PUBLIC __attribute__ ((visibility ("default")))
 
 #define START_PACKED
 #define END_PACKED __attribute__((packed))

cpp/csp/core/QueueWaiter.h

@@ -14,7 +14,7 @@ namespace csp
 
 class TimeDelta;
 
-class CSP_PUBLIC QueueWaiter
+class CSPIMPL_EXPORT QueueWaiter
 {
 public:
     QueueWaiter() : m_eventsPending( false )

cpp/csp/core/TaggedPointerUnion.h

@@ -26,7 +26,7 @@ struct IndexOf<T, TypeList<TOther, Ts...> >
 
 
 template<typename... Ts>
-class CSP_PUBLIC TaggedPointerUnion
+class CSPIMPL_EXPORT TaggedPointerUnion
 {
 public:
     static inline constexpr size_t NUM_TAGS = sizeof...(Ts);

cpp/csp/core/Time.h

@@ -21,7 +21,7 @@ const int64_t NANOS_PER_SECOND      = 1000000000;
 const int64_t SECONDS_PER_DAY       = 86400;
 const int64_t NANOS_PER_DAY         = NANOS_PER_SECOND * SECONDS_PER_DAY;
 
-class CSP_PUBLIC TimeDelta
+class CSPIMPL_EXPORT TimeDelta
 {
 public:
     constexpr TimeDelta() : TimeDelta( TimeDelta::NONE() ) {}
@@ -165,7 +165,7 @@ inline std::ostream & operator <<( std::ostream &os, const TimeDelta & d )
     return os;
 }
 
-class CSP_PUBLIC Date
+class CSPIMPL_EXPORT Date
 {
 public:
     Date() : Date( NONE() ) {}
@@ -316,7 +316,7 @@ inline std::ostream & operator <<( std::ostream &os, const Date & d )
     return os;
 }
 
-class CSP_PUBLIC Time
+class CSPIMPL_EXPORT Time
 {
 public:
     Time() : Time( -1 ) {} //NONE
@@ -446,7 +446,7 @@ inline std::ostream & operator <<( std::ostream &os, const Time & t )
 
 // Time is internally stored as an int64_t nanoseconds since 1970. 
 // All DateTime objects are stored as UTC and should be treated as such
-class CSP_PUBLIC DateTime
+class CSPIMPL_EXPORT DateTime
 {
 public:
     DateTime() : DateTime( DateTime::NONE() ) {}
@@ -597,7 +597,7 @@ inline std::ostream & operator <<( std::ostream &os, const DateTime & dt )
 //Helper class to extract day/month/year/etc info from raw timestamp
 //ie DateTimeEx dte( existingDt )
 //dte.day, etc etc
-class CSP_PUBLIC DateTimeEx : public DateTime
+class CSPIMPL_EXPORT DateTimeEx : public DateTime
 {
 public:
     DateTimeEx( const DateTime & dt );

cpp/csp/engine/AdapterManager.h

@@ -17,7 +17,7 @@ class AdapterManager;
 
 class Engine;
 
-class CSP_PUBLIC ManagedSimInputAdapter : public InputAdapter
+class CSPIMPL_EXPORT ManagedSimInputAdapter : public InputAdapter
 {
 public:
     ManagedSimInputAdapter( csp::Engine *engine, const CspTypePtr &type, AdapterManager *manager, PushMode pushMode );
@@ -94,7 +94,7 @@ bool ManagedSimInputAdapter::pushNullTick()
     return true;
 }
 
-class CSP_PUBLIC AdapterManager : public EngineOwned
+class CSPIMPL_EXPORT AdapterManager : public EngineOwned
 {
 public:
     AdapterManager( csp::Engine * );

cpp/csp/engine/AlarmInputAdapter.h

@@ -9,7 +9,7 @@ namespace csp
 {
 
 template<typename T>
-class CSP_PUBLIC AlarmInputAdapter final : public InputAdapter
+class CSPIMPL_EXPORT AlarmInputAdapter final : public InputAdapter
 {
 public:
     AlarmInputAdapter( Engine * engine, CspTypePtr & type ) : InputAdapter( engine, type, PushMode::NON_COLLAPSING )

cpp/csp/engine/BasketInfo.h

@@ -13,7 +13,7 @@ namespace csp
 
 class Node;
 
-class CSP_PUBLIC InputBasketInfo
+class CSPIMPL_EXPORT InputBasketInfo
 {
     using TickedInputs = std::vector<INOUT_ELEMID_TYPE>;
 
@@ -189,7 +189,7 @@ class CSP_PUBLIC InputBasketInfo
     bool                        m_isDynamic;
 };
 
-class CSP_PUBLIC DynamicInputBasketInfo : public InputBasketInfo
+class CSPIMPL_EXPORT DynamicInputBasketInfo : public InputBasketInfo
 {
 public:
     using ChangeCallback = std::function<void(const DialectGenericType & key,bool added, int64_t elemId, int64_t replaceId )>;
@@ -228,7 +228,7 @@ class CSP_PUBLIC DynamicInputBasketInfo : public InputBasketInfo
     TimeDelta         m_timeWindowPolicy;
 };
 
-class CSP_PUBLIC OutputBasketInfo
+class CSPIMPL_EXPORT OutputBasketInfo
 {
 public:
     OutputBasketInfo( CspTypePtr & type, Node * node, size_t size, bool isDynamic = false );
@@ -255,7 +255,7 @@ class CSP_PUBLIC OutputBasketInfo
     bool                   m_isDynamic;
 };
 
-class CSP_PUBLIC DynamicOutputBasketInfo : public OutputBasketInfo
+class CSPIMPL_EXPORT DynamicOutputBasketInfo : public OutputBasketInfo
 {
 public:
     DynamicOutputBasketInfo( CspTypePtr & type, Node * node );

cpp/csp/engine/Consumer.h

@@ -14,7 +14,7 @@ namespace csp
 class InputAdapter;
 
 //Base of either regular Nodes or output adapters
-class CSP_PUBLIC Consumer
+class CSPIMPL_EXPORT Consumer
 {
 public:
     Consumer( Engine * );

cpp/csp/engine/CppNode.h

@@ -13,7 +13,7 @@ namespace csp
 
 //CppNode is used specifically for C++ defined Nodes, and should only be used
 //for definig c++ nodes using the macros defined at the end
-class CSP_PUBLIC CppNode : public csp::Node
+class CSPIMPL_EXPORT CppNode : public csp::Node
 {
 public:
     using Shape = std::variant<std::uint64_t,std::vector<std::string>>;

cpp/csp/engine/CspEnum.h

@@ -15,7 +15,7 @@ namespace csp
 
 class CspEnumMeta;
 
-class CSP_PUBLIC CspEnumInstance
+class CSPIMPL_EXPORT CspEnumInstance
 {
 public:
     CspEnumInstance( std::string name, int64_t value, csp::CspEnumMeta * meta ) : m_name( name ), m_value( value ), m_meta( meta ) {}
@@ -35,7 +35,7 @@ class CSP_PUBLIC CspEnumInstance
 
 //As an optimization we do NOT attach meta or value to every instance of an enum.  Instead, the enum
 //holds only a pointer to a singleton CspEnumInstance, which holds the value, name, and meta pointer.
-class CSP_PUBLIC CspEnum
+class CSPIMPL_EXPORT CspEnum
 {
 public:
     CspEnum();
@@ -59,7 +59,7 @@ class CSP_PUBLIC CspEnum
 
 std::ostream &operator<<( std::ostream &os, const CspEnum & rhs );
 
-class CSP_PUBLIC CspEnumMeta
+class CSPIMPL_EXPORT CspEnumMeta
 {
 public:
     using ValueDef = std::unordered_map<std::string,int64_t>;

cpp/csp/engine/CspType.h

@@ -15,7 +15,7 @@ namespace csp
 
 class CspStringType;
 
-class CSP_PUBLIC CspType
+class CSPIMPL_EXPORT CspType
 {
 public:
 
@@ -116,7 +116,7 @@ class CSP_PUBLIC CspType
     Type m_type;
 };
 
-class CSP_PUBLIC CspStringType : public CspType
+class CSPIMPL_EXPORT CspStringType : public CspType
 {
 public:
     CspStringType(bool isBytes)
@@ -135,7 +135,7 @@ class CspEnum;
 
 class CspEnumMeta;
 
-class CSP_PUBLIC CspEnumType : public CspType
+class CSPIMPL_EXPORT CspEnumType : public CspType
 {
 public:
     CspEnumType( std::shared_ptr<CspEnumMeta> & meta ) : CspType( CspType::Type::ENUM ),
@@ -155,7 +155,7 @@ using StructPtr = TypedStructPtr<Struct>;
 
 class StructMeta;
 
-class CSP_PUBLIC CspStructType : public CspType
+class CSPIMPL_EXPORT CspStructType : public CspType
 {
 public:
     CspStructType( const std::shared_ptr<StructMeta> & meta ) : CspType( CspType::Type::STRUCT ),
@@ -168,7 +168,7 @@ class CSP_PUBLIC CspStructType : public CspType
     std::shared_ptr<StructMeta> m_meta;
 };
 
-class CSP_PUBLIC CspArrayType : public CspType
+class CSPIMPL_EXPORT CspArrayType : public CspType
 {
 public:
     CspArrayType( CspTypePtr elemType, bool isPyStructFastList = false ) :

cpp/csp/engine/CycleStepTable.h

@@ -12,7 +12,7 @@ namespace csp
 class Consumer;
 class Profiler;
 
-class CSP_PUBLIC CycleStepTable
+class CSPIMPL_EXPORT CycleStepTable
 {
 public:
     CycleStepTable();

cpp/csp/engine/Dictionary.h

@@ -23,7 +23,7 @@ using DictionaryPtr = std::shared_ptr<Dictionary>;
 class StructMeta;
 using StructMetaPtr = std::shared_ptr<StructMeta>;
 
-class CSP_PUBLIC Dictionary
+class CSPIMPL_EXPORT Dictionary
 {
 public:
     struct Data;

cpp/csp/engine/DynamicEngine.h

@@ -10,7 +10,7 @@
 namespace csp
 {
 
-class CSP_PUBLIC DynamicEngine final : public Engine
+class CSPIMPL_EXPORT DynamicEngine final : public Engine
 {
 public:
     using ShutdownFn = std::function<void()>;

cpp/csp/engine/DynamicNode.h

@@ -18,7 +18,7 @@ class DynamicEngine;
 //external inputs and triggering a cycle step.  
 //All dynamic engine input adapters as well as all external timeseries wired into dynamic are registered as inputs
 //into the DynanicNode instance so that it can invoke cycle step at the appropriate times.
-class CSP_PUBLIC DynamicNode final : public Node
+class CSPIMPL_EXPORT DynamicNode final : public Node
 {
 public:
     using Outputs = std::vector<const TimeSeriesProvider *>;

cpp/csp/engine/Engine.h

@@ -24,7 +24,7 @@ class TimeSeriesProvider;
 class Engine;
 class RootEngine;
 
-struct CSP_PUBLIC EngineOwned
+struct CSPIMPL_EXPORT EngineOwned
 {
     //force objects derived from EngineOwned to go through engine createOwnedObject methods
     //little trick here to force all derivations to fail
@@ -35,7 +35,7 @@ struct CSP_PUBLIC EngineOwned
     void operator delete( void* ptr )            { ::operator delete(ptr); }
 };
 
-class CSP_PUBLIC Engine
+class CSPIMPL_EXPORT Engine
 {
     using AdapterManagers   = std::vector<std::shared_ptr<AdapterManager>>;
     using InputAdapters     = std::vector<std::unique_ptr<InputAdapter>>;