<3​21bob321:monero.social> * Thanks Sleepy Joe

<s​yntheticbird:monero.social> Joe joe, can you pardon kewbit please

Hi, we are trying to migrate from monero-wallet-rpc to native wallet2 bindings (using monero_c, a C wrapper around wallet2). I'm trying to move from a HTTP call to check_tx_key to a C call to MONERO_Wallet_checkTxKey. However I'm a bit unsure why there is a difference between the two. check_tx_key takes a txid, tx_key and address and gives me back

confirmations and the received amount. MONERO_Wallet_checkTxKey however takes all 5 and only returns a boolean?

Probably a mistake in the monero_c wrapper code? MONERO_Wallet_checkTxKey should take a reference instead of value?

<s​yntheticbird:monero.social> probably better to ask in #monero-dev-community since Czarnek which i believe to be the author is there

jeffro256: can you take a look at monero-project/monero #9459#issuecomment-2589519946 again?

Suppose I wanted to build on top of monero-cpp and build a wrapper. Which should I use? simplewallet.h, wallet2.h or wallet2_api.h?

by monero-cpp I mean the normal monero core repo

<w​oodser:monero.social> would recommend wallet2.h for now

<w​oodser:monero.social> wallet2_api.h was created for the gui, though I hear they're working on porting all functionality to that

<w​oodser:monero.social> simplewallet.h is for simple command line tool

wallet2_api is a gratuitous extra api for no good reason (apart from thr async stuff it adds, which does add value).

But the redefinition of structures is just... wtf...

Is wallet2_api.h being used in the GUI yet? We are basically doing what the GUI is doin (wallet functionality and a few other things) but in Rust

It is.

As far as I understood wallet2_api.h is going to act as a compatability layer for the upcoming hardfork?

     74 |   SL(boost::asio::io_service::work);

        |      ~~~~~^

  /Users/me/code/monero-wrapper/monero-cpp/src/debug_utilities/object_sizes.cpp:62:39: note: expanded from macro 'SL'

     62 | #define SL(type) sl.add(#type, sizeof(type))

Anyone know how to fix this build error?

<r​brunner7:monero.social> The "Wallet API completion PR" is quite advanced already and ready for review: monero-project/monero #9464

<r​brunner7:monero.social> I am well aware that the move from wallet2 to the Wallet API does not make everybody sing with joy, but hey, still cementing the monster that is wallet2 further, in 2025? Come on :)

<r​brunner7:monero.social> Especially when you more or less have free choice, like binarybaron2 may have now

<r​brunner7:monero.social> "As far as I understood wallet2_api.h is going to act as a compatability layer for the upcoming hardfork?" Don't think so, not yet for the very next hardfork.

<r​brunner7:monero.social> The Seraphis Wallet Workgroup had the very ambitious and ultimately failed plan to replace wallet2 not by the Wallet API, but something brand new, with the goal to remove wallet2 already with the hardfork to Seraphis. Didn't get very far ...

<j​effro256:monero.social> selsta: yes, sorry I meant to earlier, but got distracted