Remove redundant insert method in serialize.

This seems to cause problems on recent clang, and looks totally
redundant and unused.

The const_iterator version is identical to the vector::const_iterator
one (which is a typedef thereof). Marking it private (instead of
removing) compiles fine, so this version is effectively unused even.
Merge pull request #3174 from TheBlueMatt/master

Re-enable Jenkins and run more tests on Jenkins than pull-tester gets
Merge pull request #3175 from TheBlueMatt/master

Missed RPC port
qt: make receive coins tab look more consistent with send coins tab

Use a GridLayout instead of a FormLayout. This makes the alignment
of the labels consistent between the tabs.
Generalize the remove-outputs check for fully-prunable transactions.

Instead of explicitly testing for the presence of any output, and
dealing with this case specially, just interpret it as an empty
CCoins.

The case previously caught using the HaveCoins check, is now handled
by the generic outs != outsBlock test.
Merge pull request #3163 from sipa/allunspendable

Generalize the remove-outputs check for fully-prunable transactions.
Merge pull request #3162 from sipa/saninsert

Remove redundant insert method in serialize
bs