Filter listaddressgroupings

[mb300sd]

My old patch for coin control converted to a pull.

• Added ability to filter empty groups and empty address from listaddressgroupings output.
• Output an object that's easier to work with than an array of different types.

[BitcoinPullTester]

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/d9ab371f837476f863ca5c7a96e52f7f052f8246 for binaries and test log.

[jgarzik]

Your output is superior to the existing output. However, I am forced to NAK because it breaks the existing API.

Given the lack of interest in this pull request, inclined to close for lack of interest plus API issue.

[luke-jr]

@jgarzik How does this break the existing API? From a readover, it seems to behave the same when called without options...?

[luke-jr]

@mb300sd BTW, this needs a rebase.

[BitcoinPullTester]

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/4c4c4d79046cc6523fcfa300485146a2bfead319 for binaries and test log.
This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/
Contact BlueMatt on freenode if something looks broken.

[gavinandresen]

NACK: jgarzik is correct, this changes the RPC API, which is unacceptable:

Output of listaddressgroupings before:

    [
        [
            "miD5tprKXUrU97H5fK7pKfwyTH9aDgi5vd",
            50.00070000
        ]
    ]

After:

    [
        {
            "address" : "miD5tprKXUrU97H5fK7pKfwyTH9aDgi5vd",
            "balance" : 50.00070000
        }
    ]

Closing. Please open a new pull request if you come up with a backwards-compatible solution.