yarp

yarp copied to clipboard

The `IWrapper::attach()` method accepts only a `PolyDriver`. There are acceptable use cases when you have a pointer to a `DeviceDriver` and you want to attach another device on that. `PolyDriver`...

drdanz

The following methods use `int` * `yarp::os::Sound::getFrequency()` * `yarp::os::Sound::setFrequency()` The following methods use `size_t` * `yarp::sig::soundfilters::resample()`

drdanz

The `yarp::os::Value` constructor ``` Value::Value(void* data, int length) ``` should be instead ``` Value::Value(void* data, size_t length) ```

drdanz

Should Subscriber::read() be const?

1

In my application, I cannot call `yarp::os::Subscriber::read()` method inside a `const` method, e.g: ```c class test { public: yarp::os::Subscriber sub; //I need to mark sub as mutable void getter(mytype& data)...

randaz81

The following methods should return `size_t` instead of `int`: * `IFrameGrabber::getRawBufferSize` * `IFrameGrabber::height` * `IFrameGrabber::width` * `IFrameGrabberRgb::height` * `IFrameGrabberRgb::width` * `IFrameGrabberImage::height` * `IFrameGrabberImage::width` * `IFrameGrabberImageRaw::height` * `IFrameGrabberImageRaw::width` There might be...

drdanz

This ends up also in `BufferedPort`: ```diff -int yarp::os::TypedReader::getPendingReads() +size_t yarp::os::TypedReader::getPendingReads() ```

drdanz

At the moment, all the callback mechanisms in YARP require writing a class that implement some interface, just to have a method that is called as callback. It would be...

drdanz

BufferedPort has method isClosed() and Port has method isOpen()

3

The title is quite explanatory of the API issue :sweat_smile:

Nicogene

Exploiting JSON in Yarp

6

This is a feature request and some brain storming to exploit better the JSON in Yarp. JSON is already used in Yarp for calling RPC command using HTTP carrier. I...

apaikan

Use enum class instead of enum

2

Replace `enum` with `enum class` in `Image` and `PointCloud`. This will cause the enforcing on the type, but it will break the API.

Nicogene