Skip to content

Commit

Permalink
display: rename DisplayBufferRef to DisplayRef
Browse files Browse the repository at this point in the history
  • Loading branch information
ayufan committed Jun 25, 2023
1 parent 0069aeb commit 6936ea8
Show file tree
Hide file tree
Showing 15 changed files with 16 additions and 24 deletions.
2 changes: 1 addition & 1 deletion esphome/components/addressable_light/display.py
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,6 @@ async def to_code(config):

if CONF_LAMBDA in config:
lambda_ = await cg.process_lambda(
config[CONF_LAMBDA], [(display.DisplayBufferRef, "it")], return_type=cg.void
config[CONF_LAMBDA], [(display.DisplayRef, "it")], return_type=cg.void
)
cg.add(var.set_writer(lambda_))
5 changes: 3 additions & 2 deletions esphome/components/display/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,11 @@
IS_PLATFORM_COMPONENT = True

display_ns = cg.esphome_ns.namespace("display")
Display = display_ns.class_("Display")
DisplayBuffer = display_ns.class_("DisplayBuffer")
DisplayPage = display_ns.class_("DisplayPage")
DisplayPagePtr = DisplayPage.operator("ptr")
DisplayBufferRef = DisplayBuffer.operator("ref")
DisplayRef = Display.operator("ref")
DisplayPageShowAction = display_ns.class_("DisplayPageShowAction", automation.Action)
DisplayPageShowNextAction = display_ns.class_(
"DisplayPageShowNextAction", automation.Action
Expand Down Expand Up @@ -96,7 +97,7 @@ async def setup_display_core_(var, config):
pages = []
for conf in config[CONF_PAGES]:
lambda_ = await cg.process_lambda(
conf[CONF_LAMBDA], [(DisplayBufferRef, "it")], return_type=cg.void
conf[CONF_LAMBDA], [(DisplayRef, "it")], return_type=cg.void
)
page = cg.new_Pvariable(conf[CONF_ID], lambda_)
pages.append(page)
Expand Down
9 changes: 0 additions & 9 deletions esphome/components/display/display.h
Original file line number Diff line number Diff line change
Expand Up @@ -133,12 +133,10 @@ enum DisplayRotation {
};

class Display;
class DisplayBuffer;
class DisplayPage;
class DisplayOnPageChangeTrigger;

using display_writer_t = std::function<void(Display &)>;
using display_buffer_writer_t = std::function<void(DisplayBuffer &)>;

#define LOG_DISPLAY(prefix, type, obj) \
if ((obj) != nullptr) { \
Expand Down Expand Up @@ -408,10 +406,6 @@ class Display {

/// Internal method to set the display writer lambda.
void set_writer(display_writer_t &&writer);
void set_writer(display_buffer_writer_t writer) {
// Temporary mapping to be removed once all lambdas are changed to use `display.DisplayRef`
this->set_writer([writer](Display &display) { return writer((display::DisplayBuffer &) display); });
}

void show_page(DisplayPage *page);
void show_next_page();
Expand Down Expand Up @@ -496,9 +490,6 @@ class Display {
class DisplayPage {
public:
DisplayPage(display_writer_t writer);
// Temporary mapping to be removed once all lambdas are changed to use `display.DisplayRef`
DisplayPage(display_buffer_writer_t writer)
: DisplayPage([writer](Display &display) { return writer((display::DisplayBuffer &) display); }) {}
void show();
void show_next();
void show_prev();
Expand Down
2 changes: 1 addition & 1 deletion esphome/components/ili9xxx/display.py
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ async def to_code(config):

if CONF_LAMBDA in config:
lambda_ = await cg.process_lambda(
config[CONF_LAMBDA], [(display.DisplayBufferRef, "it")], return_type=cg.void
config[CONF_LAMBDA], [(display.DisplayRef, "it")], return_type=cg.void
)
cg.add(var.set_writer(lambda_))

Expand Down
2 changes: 1 addition & 1 deletion esphome/components/inkplate6/display.py
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ async def to_code(config):

if CONF_LAMBDA in config:
lambda_ = await cg.process_lambda(
config[CONF_LAMBDA], [(display.DisplayBufferRef, "it")], return_type=cg.void
config[CONF_LAMBDA], [(display.DisplayRef, "it")], return_type=cg.void
)
cg.add(var.set_writer(lambda_))

Expand Down
2 changes: 1 addition & 1 deletion esphome/components/pcd8544/display.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,6 @@ async def to_code(config):

if CONF_LAMBDA in config:
lambda_ = await cg.process_lambda(
config[CONF_LAMBDA], [(display.DisplayBufferRef, "it")], return_type=cg.void
config[CONF_LAMBDA], [(display.DisplayRef, "it")], return_type=cg.void
)
cg.add(var.set_writer(lambda_))
2 changes: 1 addition & 1 deletion esphome/components/ssd1306_base/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -96,6 +96,6 @@ async def setup_ssd1306(var, config):
cg.add(var.init_invert(config[CONF_INVERT]))
if CONF_LAMBDA in config:
lambda_ = await cg.process_lambda(
config[CONF_LAMBDA], [(display.DisplayBufferRef, "it")], return_type=cg.void
config[CONF_LAMBDA], [(display.DisplayRef, "it")], return_type=cg.void
)
cg.add(var.set_writer(lambda_))
2 changes: 1 addition & 1 deletion esphome/components/ssd1322_base/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,6 @@ async def setup_ssd1322(var, config):
cg.add(var.set_external_vcc(config[CONF_EXTERNAL_VCC]))
if CONF_LAMBDA in config:
lambda_ = await cg.process_lambda(
config[CONF_LAMBDA], [(display.DisplayBufferRef, "it")], return_type=cg.void
config[CONF_LAMBDA], [(display.DisplayRef, "it")], return_type=cg.void
)
cg.add(var.set_writer(lambda_))
2 changes: 1 addition & 1 deletion esphome/components/ssd1325_base/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,6 @@ async def setup_ssd1325(var, config):
cg.add(var.set_external_vcc(config[CONF_EXTERNAL_VCC]))
if CONF_LAMBDA in config:
lambda_ = await cg.process_lambda(
config[CONF_LAMBDA], [(display.DisplayBufferRef, "it")], return_type=cg.void
config[CONF_LAMBDA], [(display.DisplayRef, "it")], return_type=cg.void
)
cg.add(var.set_writer(lambda_))
2 changes: 1 addition & 1 deletion esphome/components/ssd1327_base/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,6 @@ async def setup_ssd1327(var, config):
cg.add(var.init_brightness(config[CONF_BRIGHTNESS]))
if CONF_LAMBDA in config:
lambda_ = await cg.process_lambda(
config[CONF_LAMBDA], [(display.DisplayBufferRef, "it")], return_type=cg.void
config[CONF_LAMBDA], [(display.DisplayRef, "it")], return_type=cg.void
)
cg.add(var.set_writer(lambda_))
2 changes: 1 addition & 1 deletion esphome/components/ssd1331_base/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,6 @@ async def setup_ssd1331(var, config):
cg.add(var.init_brightness(config[CONF_BRIGHTNESS]))
if CONF_LAMBDA in config:
lambda_ = await cg.process_lambda(
config[CONF_LAMBDA], [(display.DisplayBufferRef, "it")], return_type=cg.void
config[CONF_LAMBDA], [(display.DisplayRef, "it")], return_type=cg.void
)
cg.add(var.set_writer(lambda_))
2 changes: 1 addition & 1 deletion esphome/components/ssd1351_base/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,6 @@ async def setup_ssd1351(var, config):
cg.add(var.init_brightness(config[CONF_BRIGHTNESS]))
if CONF_LAMBDA in config:
lambda_ = await cg.process_lambda(
config[CONF_LAMBDA], [(display.DisplayBufferRef, "it")], return_type=cg.void
config[CONF_LAMBDA], [(display.DisplayRef, "it")], return_type=cg.void
)
cg.add(var.set_writer(lambda_))
2 changes: 1 addition & 1 deletion esphome/components/st7735/display.py
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ async def setup_st7735(var, config):
cg.add(var.set_reset_pin(reset))
if CONF_LAMBDA in config:
lambda_ = await cg.process_lambda(
config[CONF_LAMBDA], [(display.DisplayBufferRef, "it")], return_type=cg.void
config[CONF_LAMBDA], [(display.DisplayRef, "it")], return_type=cg.void
)
cg.add(var.set_writer(lambda_))

Expand Down
2 changes: 1 addition & 1 deletion esphome/components/st7789v/display.py
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ async def to_code(config):

if CONF_LAMBDA in config:
lambda_ = await cg.process_lambda(
config[CONF_LAMBDA], [(display.DisplayBufferRef, "it")], return_type=cg.void
config[CONF_LAMBDA], [(display.DisplayRef, "it")], return_type=cg.void
)
cg.add(var.set_writer(lambda_))

Expand Down
2 changes: 1 addition & 1 deletion esphome/components/waveshare_epaper/display.py
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ async def to_code(config):

if CONF_LAMBDA in config:
lambda_ = await cg.process_lambda(
config[CONF_LAMBDA], [(display.DisplayBufferRef, "it")], return_type=cg.void
config[CONF_LAMBDA], [(display.DisplayRef, "it")], return_type=cg.void
)
cg.add(var.set_writer(lambda_))
if CONF_RESET_PIN in config:
Expand Down

0 comments on commit 6936ea8

Please sign in to comment.