pico-ssd1306

pico-ssd1306 copied to clipboard

Potential memory leak for the buffer in FrameBuffer

1

The `buffer` in the `FrameBuffer` is allocated but never freed. This could cause memory leak if the `SSD1306` is out of scope or a new buffer is set. Granted this...

zli117

#pragma once and #ifdef/#define are redundant

2

Every include file starts with `#pragma once` and continues with `#ifndef FILENAME_H` and `#define FILENAME_H` The two methods do fundamentally the same thing. I would remove the #ifndef / #define...

FunMiles

Unexpected full ascii chars

6

Hello, I've just tested the full ascii chars feature. I tried to display following characters: `\xe8 \xe9 \xea \xeb` This should result to the following display: `è é ê ë`...

sebastienroy

CMakeLists.txt changes to build as self-contained library

2

Modified CMakeLists.txt for the project to be built as a self contained library. This allows it to be included in the main project CMakeLists.txt without having to include the pico-sdk...

PDBeal

The SSD1306 constructor should not require duplicated information and bleeding of details (i.e., the I²C address) into the application code. The documentation shows an inefficient use to construct the SSD1306...

drepper