Skip to content

fix compiler warning about binary byte syntax#28435

Open
kpishere wants to merge 1 commit into
MarlinFirmware:bugfix-2.1.xfrom
kpishere:bugfix-2.1.x-binary
Open

fix compiler warning about binary byte syntax#28435
kpishere wants to merge 1 commit into
MarlinFirmware:bugfix-2.1.xfrom
kpishere:bugfix-2.1.x-binary

Conversation

@kpishere

Copy link
Copy Markdown
Contributor

Description

fix compiler warning about binary byte syntax

Requirements

changes affect dogm LCD only. there is no actual change in values, merely syntax for B00000000 to 0b00000000 to avoid a compiler style warning.

Benefits

Only improves compiling experiance.

Configurations

Test is compiling with dogm LCD configured and not receiving compiler warnings. When running, there is no change in images.

Related Issues

no related issues.

@thinkyhead

Copy link
Copy Markdown
Member

What is the compiler warning? We can fix it with a #pragma rather than make a sweeping change to this (admittedly odd) convention.

@ellensp

ellensp commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

I've only seen this on the RP2040 (the user is using an RP2040).

Marlin/src/lcd/dogm/dogm_Bootscreen.h:70:5: warning: 'B00011111' is deprecated: use 0b00011111 instead [-Wdeprecated-declarations]
   70 |     B00011111,B11111111,B11111111,B11111111,B11111111,B11111111,B11111111,
      |     ^~~~~~~~~
In file included from .platformio/packages/framework-arduino-mbed/cores/arduino/api/ArduinoAPI.h:26,
                 from .platformio/packages/framework-arduino-mbed/cores/arduino/Arduino.h:27,
                 from Marlin/src/lcd/dogm/u8g/../../../inc/../HAL/../HAL/RP2040/../shared/Marduino.h:36,
                 from Marlin/src/lcd/dogm/u8g/../../../inc/../HAL/../HAL/RP2040/../shared/HAL_SPI.h:29,
                 from Marlin/src/lcd/dogm/u8g/../../../inc/../HAL/../HAL/RP2040/HAL.h:33,
                 from Marlin/src/lcd/dogm/u8g/../../../inc/../HAL/HAL.h:30,
                 from Marlin/src/lcd/dogm/u8g/../../../inc/MarlinConfigPre-5-post.h:29,
                 from Marlin/src/lcd/dogm/u8g/../../../inc/Conditionals-5-post.h:32,
                 from Marlin/src/lcd/dogm/u8g/../../../inc/MarlinConfigPre-6-type.h:24,
                 from Marlin/src/lcd/dogm/u8g/../../../inc/Conditionals-6-type.h:33,
                 from Marlin/src/lcd/dogm/u8g/../../../inc/MarlinConfig.h:41,
                 from Marlin/src/lcd/dogm/u8g/HAL_LCD_class_defines.h:24,
                 from Marlin/src/lcd/dogm/marlinui_DOGM.h:31,
                 from Marlin/src/lcd/dogm/marlinui_DOGM.cpp:42:
.platformio/packages/framework-arduino-mbed/cores/arduino/api/Binary.h:195:3: note: declared here
  195 |   B00011111 DEPRECATED(0b00011111) = 31,
      |   ^~~~~~~~~

Binary.h is from https://github.com/arduino/ArduinoCore-API/blob/master/api/Binary.h

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants