Show the receiving address for generation transactions #417

As has been requested by a few people, here is a patch that shows the address coins were generated to in transaction dumps (listtransaction, etc).

Nice! Maybe this should also be added in the UI (Generated with: address).

Removing the return in wallet.cpp is harmless, but it has nothing to do with this issue. Better to remove it in order to prevent merge conflicts.

It is required to remove the return statement. If the rest of the function does not execute, listReceived won't be filled with any data, and therefore the receiving address will not be available to ListTransactions.

Ok, I'm confused. I misread the diff.

But there is a problem with that, because when listReceived is filled the transaction will appear twice: once with category "generated" and once with category "received".

Fixed, I hope. :)

In fact, generated amounts could be handled internally exactly like received amounts. Then, It would be trivial to associate them with their respective addresses and accounts in the listing (reusing existing code). In particular, this would also allow us to deal with non-standard (multiple address) generations properly.

I came up with a8e6510042f2f7f951414c7da124668b30da4acd. Interestingly, this is both a code simplification and a fix for a lacking feature (almost bug). I think the "few people" you mentioned would happy with such a change, would they?

@sgimenez: Looks good. I'll close this one since your patch supersedes it (and this patch was always a hack anyway).