X-CASH-official / xcash-core

📦 First Cryptonote coin with public & private transactions, custom DPOS consensus
https://xcash.foundation
Other
66 stars 20 forks source link

[bug] annoying debug build bug #27

Closed ghost closed 2 years ago

ghost commented 2 years ago

🐛 BUG REPORT

Expected Behavior The debug build command make debug --jobs=$(nproc) should build successfully with no changes after cloning the build tree.

Current Behavior The debug build errors out at xcash-core/src/debug_utilities/object_sizes.cpp, not recognizing 'X-CASH'. Likely a global search/replace residual.

Possible Solution Lines 112-117 should be changed from:

  SL(X-CASH::WalletImpl);
  SL(X-CASH::AddressBookRow);
  SL(X-CASH::TransactionInfoImpl);
  SL(X-CASH::TransactionHistoryImpl);
  SL(X-CASH::PendingTransactionImpl);
  SL(X-CASH::UnsignedTransactionImpl);

to

  SL(XCash::WalletImpl);
  SL(XCash::AddressBookRow);
  SL(XCash::TransactionInfoImpl);
  SL(XCash::TransactionHistoryImpl);
  SL(XCash::PendingTransactionImpl);
  SL(XCash::UnsignedTransactionImpl);

to match the correct XCash namespace.

Steps to Reproduce (for bugs) type 'make debug'

Context Having a debug build fail is annoying. :)

Your Environment Ubuntu 20.04

zachhildreth commented 2 years ago

Thanks @CygnusMiner for finding this!

The fix provided above did work, and I will include this in the next testnet update