Potential memory leak for the buffer in FrameBuffer

[zli117]

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 may not be a very big issue since in most of the use cases SSD1306 lifespan will be the whole program lifespan. It's just that personally I think this still should be handled properly. Alternatively, instead of allocating the buffer on heap, just put it on stack, as the buffer size will always be 1024, and keep a member variable to point to member buffer or user-set buffer.

[Harbys]

Oh yea I never considered this and assumed everyone would want their screens connected over entire program lifespan like you said. Definitely something to fix