Skip to content

Commit 5e6c86a

Browse files
Merge #6142: fix: build with -Wdocumentation - rename param pwallet to wallet
bebf391 fix: build with -Wdocumentation - rename param pwallet to wallet (Konstantin Akimov) Pull request description: ## Issue being fixed or feature implemented CI failures like that: https://gitlab.com/dashpay/dash/-/jobs/7400661581 It happened due to 2 PRs merged one after each other without rebasing: - enabling -Wdocumentation in bitcoin#21631 (#6113) - renaming param while doxygen comment is forgotten in #6114 ## What was done? It contains changes from #6133 Thanks Vijay for spotting issue, this PR is DNM if 6133 will get merged faster. ## How Has This Been Tested? See CI ## Breaking Changes N/A ## Checklist: - [x] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have added or updated relevant unit/integration/functional/e2e tests - [ ] I have made corresponding changes to the documentation - [x] I have assigned this pull request to a milestone ACKs for top commit: UdjinM6: utACK bebf391 PastaPastaPasta: utACK bebf391 Tree-SHA512: 34265f09098c04593a9d52709e5b8aed8460248762655945e5c86a65adb9aa49ab6f0bdb21cd907d353fe6b10e1ff2e07d05528166bf8e2140eb2c9ea1acbd33
2 parents 5211886 + bebf391 commit 5e6c86a

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/wallet/rpcwallet.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1332,7 +1332,7 @@ static void MaybePushAddress(UniValue & entry, const CTxDestination &dest)
13321332
/**
13331333
* List transactions based on the given criteria.
13341334
*
1335-
* @param pwallet The wallet.
1335+
* @param wallet The wallet.
13361336
* @param wtx The wallet transaction.
13371337
* @param nMinDepth The minimum confirmation depth.
13381338
* @param fLong Whether to include the JSON version of the transaction.

0 commit comments

Comments
 (0)