Remaining feat count corrupted on load

Adding one standard feat save/load it distracts two from the available feats.

Thanks for the detective work! We can reproduce this now

I've spent some time tracing this in both trunk and 6.2 branch. In the 6.2 branch code, when a character is loaded, the abilities are added directly to the facet (see pcgen.core.PlayerCharacter.addAbility(Category<Ability>, Ability)). In the trunk code however, the abilities are sent through pcgen.core.AbilityUtilities.finaliseAbility(PlayerCharacter, CNAbilitySelection) which charges the pool again. Would you be able to have a look at it please?

Proposed Patch - changes how Feat Pools are calculated

Note: FEATPOOL line is now ignored, rather it is the USERPOOL line for FEAT that will control the value (overwrites the FEATPOOL line)

Note also: FEATPOOL is not currently written with the same value as 6.2
(so may need to address this, but since it's ignored...)

svn 24155