Modernized the code for GoodSeedProducer

The reader arguments were being leaked if more than one Run is processed
in the job. Switching all pointer ownership to std::unique_ptr simplified
all memory handling in the class.
Removed to unused member variables: ps1Clus, ps2Clus.
Moved header into plugin directory.
Declare plugin directly in .cc file.
NOTE: The .h and .cc file can now be merged into 1 file, but I didn't
do this to avoid making code diffs more difficult.
You are not fixing it in this PR I assume? It relates to line 512 of GoodSeedProducer.cc ("j < 9"),lines 524-34 of the same and also line 540 ("iy<81") I suppose. And maybe other parts of the code.