alpha
Login
or
Join now
jmrp.io
/
kleidos
Star
0
Fork
0
Atom
Configure Feed
Issues
Pull Requests
Commits
Tags
Feed URL
Select the types of activity you want to include in your feed.
[READ-ONLY] Mirror of https://github.com/jmrplens/kleidos. Kleidos — Hardware BLE password manager for ESP32-S3 (M5StickC Plus2, StickS3, M5Stack Gray, T-Deck, Cardputer)
jmrplens.github.io/kleidos/
Star
0
Fork
0
Atom
Configure Feed
Issues
Pull Requests
Commits
Tags
Feed URL
Select the types of activity you want to include in your feed.
Overview
Issues
Pulls
Pipelines
kleidos
/
src
/
drivers
/
bus
/
at
main
1 file
José M. Requena Plens
Fix build: add missing ; to I2cDevice class definition
5w ago
3d5623ae
i2c_device.h
Fix build: add missing ; to I2cDevice class definition The @addtogroup drv_bus closer ('/** @} */') was placed inside the class scope (4-space indent) just before the class's closing brace, which was shared with the namespace close. The brace closed the class but had no trailing ';', so the compiler reported: src/drivers/bus/i2c_device.h:135:2: error: expected ';' after class definition 135 | } // namespace kleidos::drivers::bus | ^ | ; This cascaded to 'imu' in namespace 'kleidos::drivers::bus::kleidos::drivers' in every board_cardputer.cpp / board_plus*.cpp that pulls in i2c_device.h via the TCA8418 keyboard driver (a T-Deck/Cardputer inclusion path). Fix: - Add the missing ';' to the class close ('};' instead of '}'). - Move the '@addtogroup drv_bus' closer from inside the class (4-space indent, 132-134) to its proper file-scope position (column 0, immediately before the namespace close), matching the style of every other @addtogroup wrap in src/. The closer is kept as '/** @} */' (multi-line) to match the '/** @addtogroup drv_bus / @{} */' opener 30 lines above — same style consistency rule from the round-1 group-style pass. Verified: pio run -e sticks3 now succeeds, restoring the cardputer / m5stack_gray / m5stickc_plus1 / m5stickc_plus2 / sticks3 / tdeck build matrix.
1 month ago