You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The header <mraa/types.h> purports to be for basic shared types, but it contains an i2c-specific type, mraa_i2c_mode_t. That should really be in <mraa/i2c.h> instead.
This would be equivalent to what <mraa/spi.h> does with mraa_spi_mode_t.
The text was updated successfully, but these errors were encountered:
Agreed, the struct should 'ideally' be duplicated in the C++ header and the C header so that SWIG uses the correct name. The reason for throwing it in the types.h is that swig takes that file (but not i2c.h) and will strip the prefix away (since we have the namespace/module/package name). So it really depends on whether you prefer enum duplication or things in the right place :) I'm happy to take a PR to change this.
The header
<mraa/types.h>
purports to be for basic shared types, but it contains an i2c-specific type,mraa_i2c_mode_t
. That should really be in<mraa/i2c.h>
instead.This would be equivalent to what
<mraa/spi.h>
does withmraa_spi_mode_t
.The text was updated successfully, but these errors were encountered: