clang-tidy error mbt std::deque & std::unique_ptr

[xces]

Mijn vraag
Ik heb een PR geopend voor ESPHome zodat ik mijn zwembad controller kan afmaken. Hiervoor moet ik een bestaand sensor component van EZO uitbreiden met nieuwe commando's.

Op 1 na gaan alle checks goed, maar bij 1 check geeft clang-tidy op diverse plekken nog een opmerking die ik dmv het Discord kanaal van EspHome heb kunnen vertalen naar "Je moet unique_ptr gebruiken". Helaas is de PR niet oorspronkelijk door mij gemaakt en is mijn kennis van C++ dermate slecht (zeker met pointers) dat ik niet veel verder kom. Zou er iemand met mij mee kunnen kijken en evt een oplossing aandragen?

Wat ik al gevonden of geprobeerd heb
Diverse andere implementaties geprobeerd, via Discord de developers gevraagd wat de clang error betekend maar zonder succes.

Wat ik wil bereiken:
Ik heb een queue met commando's die in sequentie moet worden uitgevoerd. Deze declareer ik hier als volgt:

std::deque<EzoCommand *> commands_;

In de loop() kijk ik of het commando wat we nu aan het uitvoeren een READ is, en zo niet dan wil ik dat na het huidige commando een read word uitgevoerd, dat zit hier en probeer ik dus op positie 2 een nieuw commando te injecteren.

Dit werkt op andere platformen maar niet op de ESP32.. Als iemand met mij mee kan kijken zou dat heel fijn zijn.

[Rmg]

Dit werkt wel alleen is een bad habit (volgens de clang-tidy ruleset)

Hier staat je probleem beschreven
https://clang.llvm.org/ex...elines-owning-memory.html

code:

std::unique_ptr<EzoCommand > commands_;

is zo uit mn hoofd hoe je je ptr moet gaan declareren. Het voordeel van smart pointers is dat ze zichzelf opruimen

[farlane]

Rmg schreef op woensdag 15 juni 2022 @ 10:06:
Dit werkt wel alleen is een bad habit (volgens de clang-tidy ruleset)

Hier staat je probleem beschreven
https://clang.llvm.org/ex...elines-owning-memory.html

code:

std::unique_ptr<EzoCommand > commands_;

is zo uit mn hoofd hoe je je ptr moet gaan declareren. Het voordeel van smart pointers is dat ze zichzelf opruimen

Ehm, dat is geen aternatief voor:

C++:

std::deque<EzoCommand*> commands_;

Dan zou je eerder een :

C++:

std::deque<std::unique_ptr<EzoCommand>> commands_;

willen. Dat heeft echter wel gevolgen voor het gebruik van die pointers.