Remove BTCPP_API export macros — redundant with CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS

The selective BTCPP_API approach only covered a few classes while the
library exports many more free functions and class members. Since
CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS already handles all symbol exports,
the partial BTCPP_API annotations were dead code. Remove export.h and
all BTCPP_API annotations to keep the diff minimal.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: facontidavide

CMakeLists.txt

@@ -218,10 +218,6 @@ endif()
 if (BTCPP_SHARED_LIBS)
     set(CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS ON)
     add_library(${BTCPP_LIBRARY} SHARED ${BT_SOURCE})
-    target_compile_definitions(${BTCPP_LIBRARY}
-        PUBLIC BTCPP_SHARED_LIBS
-        PRIVATE behaviortree_cpp_EXPORTS
-    )
 else()
     set(CMAKE_POSITION_INDEPENDENT_CODE ON)
     add_library(${BTCPP_LIBRARY} STATIC ${BT_SOURCE})

include/behaviortree_cpp/action_node.h

@@ -16,8 +16,6 @@
 
 #include "leaf_node.h"
 
-#include "behaviortree_cpp/export.h"
-
 #include <atomic>
 #include <future>
 #include <mutex>
@@ -33,7 +31,7 @@ namespace BT
  * A particular derived class is free to override executeTick() as needed.
  *
  */
-class BTCPP_API ActionNodeBase : public LeafNode
+class ActionNodeBase : public LeafNode
 {
 public:
   ActionNodeBase(const std::string& name, const NodeConfig& config);
@@ -55,7 +53,7 @@ class BTCPP_API ActionNodeBase : public LeafNode
  * explicitly prevents the status RUNNING and doesn't require
  * an implementation of halt().
  */
-class BTCPP_API SyncActionNode : public ActionNodeBase
+class SyncActionNode : public ActionNodeBase
 {
 public:
   SyncActionNode(const std::string& name, const NodeConfig& config);
@@ -87,7 +85,7 @@ class BTCPP_API SyncActionNode : public ActionNodeBase
  * Using lambdas or std::bind it is easy to pass a pointer to a method.
  * SimpleActionNode is executed synchronously and does not support halting.
  */
-class BTCPP_API SimpleActionNode : public SyncActionNode
+class SimpleActionNode : public SyncActionNode
 {
 public:
   using TickFunctor = std::function<NodeStatus(TreeNode&)>;
@@ -129,7 +127,7 @@ class BTCPP_API SimpleActionNode : public SyncActionNode
  * a TreeNode::emitWakeUpSignal() will be called.
  */
 
-class BTCPP_API ThreadedAction : public ActionNodeBase
+class ThreadedAction : public ActionNodeBase
 {
 public:
   ThreadedAction(const std::string& name, const NodeConfig& config)
@@ -172,7 +170,7 @@ using AsyncActionNode = ThreadedAction;
  *
  * -) if halted, method onHalted() is invoked
  */
-class BTCPP_API StatefulActionNode : public ActionNodeBase
+class StatefulActionNode : public ActionNodeBase
 {
 public:
   StatefulActionNode(const std::string& name, const NodeConfig& config)
@@ -209,7 +207,7 @@ class BTCPP_API StatefulActionNode : public ActionNodeBase
  * It is up to the user to decide when to suspend execution of the Action and resume
  * the parent node, invoking the method setStatusRunningAndYield().
  */
-class BTCPP_API CoroActionNode : public ActionNodeBase
+class CoroActionNode : public ActionNodeBase
 {
 public:
   CoroActionNode(const std::string& name, const NodeConfig& config);

include/behaviortree_cpp/blackboard.h

@@ -3,7 +3,6 @@
 #include "behaviortree_cpp/basic_types.h"
 #include "behaviortree_cpp/contrib/json.hpp"
 #include "behaviortree_cpp/exceptions.h"
-#include "behaviortree_cpp/export.h"
 #include "behaviortree_cpp/utils/locked_reference.hpp"
 #include "behaviortree_cpp/utils/safe_any.hpp"
 
@@ -30,7 +29,7 @@ struct StampedValue
  * @brief The Blackboard is the mechanism used by BehaviorTrees to exchange
  * typed data.
  */
-class BTCPP_API Blackboard
+class Blackboard
 {
 public:
   using Ptr = std::shared_ptr<Blackboard>;

include/behaviortree_cpp/bt_factory.h

@@ -17,7 +17,6 @@
 #include "behaviortree_cpp/behavior_tree.h"
 #include "behaviortree_cpp/contrib/json.hpp"
 #include "behaviortree_cpp/contrib/magic_enum.hpp"
-#include "behaviortree_cpp/export.h"
 
 #include <filesystem>
 #include <functional>
@@ -92,7 +91,7 @@ bool WildcardMatch(const std::string& str, StringView filter);
  * @brief Struct used to store a tree.
  * If this object goes out of scope, the tree is destroyed.
  */
-class BTCPP_API Tree
+class Tree
 {
 public:
   // a tree can contain multiple subtree.
@@ -223,7 +222,7 @@ class Parser;
  * Some node types are "builtin", whilst other are used defined and need
  * to be registered using a unique ID.
  */
-class BTCPP_API BehaviorTreeFactory
+class BehaviorTreeFactory
 {
 public:
   BehaviorTreeFactory();

include/behaviortree_cpp/condition_node.h

@@ -16,8 +16,6 @@
 
 #include "leaf_node.h"
 
-#include "behaviortree_cpp/export.h"
-
 namespace BT
 {
 /**
@@ -30,7 +28,7 @@ namespace BT
  * Conditions are typically used to check the state of the world or
  * the system before performing an action (e.g., "IsDoorOpen", "IsBatteryLow").
  */
-class BTCPP_API ConditionNode : public LeafNode
+class ConditionNode : public LeafNode
 {
 public:
   ConditionNode(const std::string& name, const NodeConfig& config);
@@ -64,7 +62,7 @@ class BTCPP_API ConditionNode : public LeafNode
  *
  * Using lambdas or std::bind it is easy to pass a pointer to a method.
  */
-class BTCPP_API SimpleConditionNode : public ConditionNode
+class SimpleConditionNode : public ConditionNode
 {
 public:
   using TickFunctor = std::function<NodeStatus(TreeNode&)>;

include/behaviortree_cpp/control_node.h

@@ -13,7 +13,6 @@
 
 #pragma once
 
-#include "behaviortree_cpp/export.h"
 #include "behaviortree_cpp/tree_node.h"
 
 #include <vector>
@@ -29,7 +28,7 @@ namespace BT
  * Each derived class implements specific rules about if, when, and how many
  * times children are ticked.
  */
-class BTCPP_API ControlNode : public TreeNode
+class ControlNode : public TreeNode
 {
 protected:
   std::vector<TreeNode*> children_nodes_;

include/behaviortree_cpp/decorator_node.h

@@ -1,7 +1,6 @@
 #ifndef DECORATORNODE_H
 #define DECORATORNODE_H
 
-#include "behaviortree_cpp/export.h"
 #include "behaviortree_cpp/tree_node.h"
 
 namespace BT
@@ -16,7 +15,7 @@ namespace BT
  * - Add timing constraints (e.g., Timeout, Delay)
  * - Conditionally execute the child (e.g., Precondition)
  */
-class BTCPP_API DecoratorNode : public TreeNode
+class DecoratorNode : public TreeNode
 {
 protected:
   TreeNode* child_node_;

include/behaviortree_cpp/export.h

@@ -15,7 +15,6 @@
 
 #include "behaviortree_cpp/basic_types.h"
 #include "behaviortree_cpp/blackboard.h"
-#include "behaviortree_cpp/export.h"
 #include "behaviortree_cpp/scripting/script_parser.hpp"
 #include "behaviortree_cpp/utils/signal.h"
 #include "behaviortree_cpp/utils/strcat.hpp"
@@ -130,7 +129,7 @@ inline constexpr bool hasNodeFullCtor()
 }
 
 /// Abstract base class for Behavior Tree Nodes
-class BTCPP_API TreeNode
+class TreeNode
 {
 public:
   typedef std::shared_ptr<TreeNode> Ptr;