Skip to content

Hurray for ssd1306_bitmaps() #19

Description

@cvonk

Thanks for the bitmaps function. I am using it!

Just ran into something that could use some attention.

In the new function

void ssd1306_bitmaps(SSD1306_t * dev, int xpos, int ypos, uint8_t * bitmap, int width, int height, bool invert)

The xpos, ypos and height parameters are in pixels. I suggest either changing the parameter width to mean width in pixels, or renaming it so that it is clear that it is pixels/8.

Also, the ImageDemo example calls

ssd1306_bitmaps(&dev, xpos, ypos, batman, 4, 12, false);

It passes the height as 12, but the image in the array batman[] is 13 pixels heigh.

/Coert

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions