Donna Whisnant

The `TwoWire::~TwoWire()` destructor should use `delete[]` and not `delete` to correspond with the `new uint8_t[bufferLength]` in the constructor. It was mostly working by accident since the TwoWire objects are generally...

The manuf:BOURNS-TC33X-2 footprint has a 1.6mm gap between the pads for pins 1 and 3. However, the Bourns datasheet shows the gap should be 0.8mm: http://www.bourns.com/docs/Product-Datasheets/TC33.pdf

WRAM double read is only needed for 32-bit values

3

I don't think it's necessary for this `vs1053::Mp3ReadWRAM` function to read the WRAM value twice. I assume the reason it's being done is because of section 10.11 in the datasheet,...

8_Compressor_RMS example project file is empty

3

The [8_Compressor_RMS](https://github.com/MCUdude/SigmaDSP/blob/3d151affd84172732a1ff82ccf50f4be9280a834/examples/8_Compressor_RMS/SigmaStudio_project/8_Compressor_RMS.dspproj) project file has zero-bytes and doesn't open in SigmaStudio.

Don't waste RAM in the low-level TWI Wire driver

2

This originated on MCUdude/MiniCore#245 Since `twi_readFrom()` and `twi_writeTo()` are both blocking functions, there is no need to allocate a special `twi_masterBuffer`. Doing so wastes valuable RAM, uses extra time to...

The prototypes in [USB_Host.h](https://github.com/arduino/Arduino/blob/master/hardware/arduino/sam/system/libsam/include/USB_host.h) for the functions from [uotghs_host.c](https://github.com/arduino/Arduino/blob/master/hardware/arduino/sam/system/libsam/source/uotghs_host.c) should be tagged as `extern "C"` since the header can be included into C++ source and is referencing C code. Without...

The version of the `addMetaEvent` function that takes the `metaData` as a `std::vector` type should accept that `std::vector` argument as a `const` type. Otherwise, callers of this function that pass...

The length checks for the `msgs` buffer, here: https://github.com/allanbian1017/i2c-ch341-usb/blob/f635589bd0f14fa0ce7f72d3f57d85a77da7a0dc/i2c-ch341-usb.c#L100 and: https://github.com/allanbian1017/i2c-ch341-usb/blob/f635589bd0f14fa0ce7f72d3f57d85a77da7a0dc/i2c-ch341-usb.c#L144-L145 are wrong and run off the end of the `out_buf` buffer if the caller passes 27 for the length...