fix: api_status.h macros colide between VW and RL

VowpalWabbit · Feb 14, 2024 · 5df10a5 · 5df10a5
1 parent dba1c45
commit 5df10a5
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/vowpalwabbit/fb_parser/include/vw/fb_parser/parse_example_flatbuffer.h b/vowpalwabbit/fb_parser/include/vw/fb_parser/parse_example_flatbuffer.h
@@ -4,7 +4,6 @@
 
 #pragma once
 
-#include "vw/core/api_status.h"
 #include "vw/core/example.h"
 #include "vw/core/multi_ex.h"
 #include "vw/core/shared_data.h"
@@ -14,7 +13,10 @@
 namespace VW
 {
 
-class api_status;
+namespace experimental
+{
+  class api_status;
+}
 
 using example_sink_f = std::function<void(VW::multi_ex&& spare_examples)>;
 

diff --git a/vowpalwabbit/fb_parser/src/parse_example_flatbuffer.cc b/vowpalwabbit/fb_parser/src/parse_example_flatbuffer.cc
@@ -4,6 +4,7 @@
 
 #include "vw/fb_parser/parse_example_flatbuffer.h"
 
+#include "vw/core/api_status.h"
 #include "vw/core/action_score.h"
 #include "vw/core/best_constant.h"
 #include "vw/core/cb.h"

diff --git a/vowpalwabbit/fb_parser/src/parse_label.cc b/vowpalwabbit/fb_parser/src/parse_label.cc
@@ -2,6 +2,7 @@
 // individual contributors. All rights reserved. Released under a BSD (revised)
 // license as described in the file LICENSE.
 
+#include "vw/core/api_status.h"
 #include "vw/core/action_score.h"
 #include "vw/core/best_constant.h"
 #include "vw/core/cb.h"