13:41:06
jpk68:matrix.org:
Is it discouraged for new core code to use Boost libraries? What is the consensus around whether Boost/epee/STL should be used for different things?
13:44:42
moneromooo:
STL is standard, and good. Use it when you can. Not super new stuff though. Boost has nice stuff STL does not. If overlap, if in doubt, use whatever the code you're modifying uses. If it uses nothing, whatever monero overall uses. If none, STL.
13:45:21
moneromooo:
Some people really dislike epee, but I would use the same logic as above really. Just don't go out of your way to use it.
13:46:04
moneromooo:
There may be special policies for the new wallet code specifically, I wouldn't know.
13:48:20
moneromooo:
And I'd try to keep down on fancy template code. It can get unmaintainable by anyone else than the writer, which isn't good.
13:49:23
jpk68:matrix.org:
Thanks :)
13:49:32
jpk68:matrix.org:
I can see why some people dislike epee, haha
17:47:58
jeffro256:monero.social:
I would say that Boost serialization specifically is discouraged in new code
18:11:08
alex_005_005:matrix.org:
Because the less wrappers, the better and cleaner the code?
18:13:20
jeffro256:monero.social:
Because it's not a great format for any specific reason, and it has almost non existent support in the broader ecosystem
18:13:56
jeffro256:monero.social:
And it's not currently used for any important functions even in our codebase besides the p2p peer list file
18:15:36
moneromooo:
boost serialization is buggy and a probable security issue. I've seen it crash, and could not understand the code. We have better anyway. It's now used only for wallet stuff IIRC, which is not attacker controlled.
23:06:57
azerx:
Hi guys, does monero core wallet support the latest trezor now?
23:07:31
azerx:
The bloody Ledger stax still resets with monero wallet so i need to stop using it with ledger
23:15:37
jpk68:matrix.org:
I believe not
23:15:39
jpk68:matrix.org:
https://github.com/monero-project/monero-gui/issues/4567#issuecomment-3921948795
23:17:51
azerx:
ok thanks, have to use local wallet i guess for now