Makes sense, utACK

Not the only one, please fix also the second one in src/rpc/mining.cpp.

I was once wondering if this should be changed to transaction pool, but I don't think it is feasible to change at this point.

Yes, in retrospect that would have been a much better name. But everyone knows what a mempool is by now, changing it will just seed more confusion. Better to go with the least invasive change.

ok- I changed that one in src/rpc/mining.cpp, the changes are on the same branch patch-7 but now I guess I should squash the commits? is it so?

Can I squash commits using the GitHub web app? If not I can do it from my console but not until later this evening.

I agree that mempool is kind of standard jargon, it's in the "Mastering Bitcoin" book by Andreas Antonopoulos for instance

Squashed the commits

utACK

On Thursday, 27 October 2016, Wladimir J. van der Laan <
notifications@github.com> wrote:

Makes sense, utACK

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#9029 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA0t8ktEH6WL21dlgOmHD2TsY-3qCbfiks5q4IHsgaJpZM4KiLCN
.