PR per driver sounds good to me.
The required change in tcs37727 indeed involves a bit more. But I hope that
everyone agrees that you cannot change a const object into a non-const
object.
Take a look at the doxygen. The param[in] is misleading, the object will
be modified.
Ha, it even says so
A controversial idea is to change the implementation to C++ and make the
driver a proper class with inheritance and utilize polymorphism with
dynamic casts.
/Joakim
Den tor 27 dec. 2018 21:51 skrev Kees Bakker :
> On 27-12-18 13:54, Kaspar Schleiser wrote:
> > Hi,
> >
> > On 12/26/18 11:16 PM,
On 27-12-18 13:54, Kaspar Schleiser wrote:
Hi,
On 12/26/18 11:16 PM, Kees Bakker wrote:
Suppose I make a Pull Request to eliminate casts, would that be picked up?
Always welcome! +1 on Joakim's hint to keep the PR's small.
Sure
void at86rf2xx_tx_exec(const at86rf2xx_t *dev)
{
netdev_t
Hi,
On 12/26/18 11:16 PM, Kees Bakker wrote:
> Suppose I make a Pull Request to eliminate casts, would that be picked up?
Always welcome! +1 on Joakim's hint to keep the PR's small.
> void at86rf2xx_tx_exec(const at86rf2xx_t *dev)
> {
> netdev_t *netdev = (netdev_t *)dev;
What would be the
Hi,
In case you are searching for us. We were switched around a little:
https://35c3.c3nav.de/l/c:0:438.09:481.89/@0,439.08,482.83,5
Kind Regards,
Martine
Am Di., 25. Dez. 2018, 21:10 hat Martine Lenders
geschrieben:
> Oops forgot the link: https://35c3.c3nav.de/l/riotos/
>
> Am Di., 25. Dez.