robotology / wearables

Data collection framework for wearable sensors
BSD 3-Clause "New" or "Revised" License
19 stars 11 forks source link

Remove unused actuators message definitions #171

Closed RiccardoGrieco closed 1 year ago

RiccardoGrieco commented 1 year ago

This PR to remove unused definitions for actuators message.

Reason

Currently, the following line will not compile:

#include <thrift/WearableData.h>
#include <thrift/WearableActuatorCommand.h>

int main()
{
    return 0;
}

With the following message:

In file included from <superbuild-dir>/build/install/include/thrift/ActuatorInfo.h:16,
                 from <superbuild-dir>/build/install/include/thrift/WearableActuatorCommand.h:16,
                 from myCode.cpp:13:
<superbuild-dir>/build/install/include/thrift/ActuatorStatus.h:21:13: error: ‘ERROR’ conflicts with a previous declaration
   21 |     ERROR = 0,
      |             ^
In file included from <superbuild-dir>/build/install/include/thrift/SensorInfo.h:16,
                 from <superbuild-dir>/build/install/include/thrift/Accelerometer.h:16,
                 from <superbuild-dir>/build/install/include/thrift/WearableData.h:16,
                 from myCode.cpp:12:
<superbuild-dir>/build/install/include/thrift/SensorStatus.h:21:5: note: previous declaration ‘wearable::msg::SensorStatus wearable::msg::ERROR’

Because of the following definitions in the thrift messages:

RiccardoGrieco commented 1 year ago

cc @CarlottaSartore