dbunit Merge Request #30: Feature #200: CachedDataSet has redundant code. (merged)

CachedDataSet did not make use of member fields and functions inherited
from AbstractDataSet. Now the code has been simplified, opening the way
for easier maintenance and extension.

IMPORTANT CHANGE: AbstractDataSet.initialize() becomes protected.

This change required that AbstractDataSet.initialize() become
'protected' instead of 'private', so that CachedDataSet could call it
instead of implementing its own duplicate version.

It also required a small renaming of the initialize() function to
initializeFlatDtdDataSet() in FlatDtdDataSet to avoid an interference.
Other classes are not affected.

WARNING:

This change is incompatible with merge request #29. You decide which one
first to merge, then I will make the required changes to the other merge
request to make things compatible.

[Jeff Jensen]

I suggest combining MR #29 and #30 since they are related and then there is no MR ordering problem.
However, this one seems good to merge as is so let me know if you prefer to not combine.

[Jeff Jensen]

Hi Hans, want to make any further progress on these? We should finish and merge them!

[Hans Deragon]

Ok. I have both FlatDtdDataSet & CacheDataSet fixed. The merge should be simple.

[Jeff Jensen]

Thanks Hans. Did the other subclasses of AbstractDataSet have these issues too? Wondering if these were the only ones or more to do yet.