Skip to content

Commit

Permalink
Remove Windows-only preprocessor macros
Browse files Browse the repository at this point in the history
This caused builds with macOS to fail, however since this is the serial driver, this isn't a meaningful change as its not maintained and slated for removal. This however does clear build failures with macOS.
  • Loading branch information
garrettsummerfi3ld committed Sep 23, 2024
1 parent ae5b796 commit bff31c8
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 12 deletions.
6 changes: 0 additions & 6 deletions src/main/native/cpp/Drivers/Serial/SerialDevice.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,6 @@
* POSSIBILITY OF SUCH DAMAGE.
*/

#ifdef _WIN32

#include "rev/Drivers/SerialPort/SerialDevice.h"

#include <iostream> //TODO: Remove
Expand Down Expand Up @@ -155,7 +153,3 @@ bool SerialDevice::IsConnected()

} // namespace usb
} // namespace rev

#else
typedef int __ISOWarning__CLEAR_;
#endif // _WIN32
6 changes: 0 additions & 6 deletions src/main/native/cpp/Drivers/Serial/SerialDriver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,6 @@
* POSSIBILITY OF SUCH DAMAGE.
*/

#ifdef _WIN32

#include "rev/Drivers/SerialPort/SerialDriver.h"
#include "rev/Drivers/SerialPort/SerialDevice.h"
#include "rev/CANBridgeUtils.h"
Expand Down Expand Up @@ -81,7 +79,3 @@ std::unique_ptr<CANDevice> SerialDriver::CreateDeviceFromDescriptor(const char*

} // namespace usb
} // namespace rev

#else
typedef int __ISOWarning__CLEAR_;
#endif // _WIN32

0 comments on commit bff31c8

Please sign in to comment.