Add address groupings RPC from coderrr’s coincontrol patches. #1642

1. 
   gmaxwell commented at 4:52 pm on August 1, 2012: contributor
2. 
   gmaxwell commented at 4:54 pm on August 1, 2012: contributor

   This is currently the unmodified listaddressgroupings RPC from the coincontrol patch. It’s been tested for basic functionality.

   I observe that it misbehaves a little when you have transaction with some inputs which aren’t yours e.g.: [ “mgrxrEzVVvf5TDt4PVc3EpiJrJGt52RXfZ”, 100.00000000, "" ]

   (the empty quote)

   But I expect thats easily fixed. The bigger question is do we want to change this to make it work better in the context of raw transactions? Listunspent gives txouts without addresses, so going from this list to a raw transaction doesn’t quite work. Ideas?

3. 
   jgarzik commented at 5:07 pm on August 1, 2012: contributor

   ACK
4. 
   gmaxwell commented at 5:14 pm on August 1, 2012: contributor

   Perhaps when it can’t decode an address it should return the txout? And the mismatch with listunspent should be fixed by allowing listunspent to take a list of addresses to filter with?
5. Add address groupings RPC from the coincontrol patches.

   Signed-off-by: Gregory Maxwell <greg@xiph.org>

   396878ff5f
6. 
   BitcoinPullTester commented at 7:12 pm on August 8, 2012: none

   Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/396878ff5fd76905156d442106ef1ae973748d00 for binaries and test log.
7. 
   gmaxwell commented at 2:42 pm on August 20, 2012: contributor

   duplicated in another pull request (#1672) now.
8. gmaxwell closed this on Aug 20, 2012

   ---

9. suprnurd referenced this in commit 8949f4345f on Dec 5, 2017
10. DrahtBot locked this on Sep 8, 2021

Contributors
gmaxwell jgarzik BitcoinPullTester