Fix StrippingCache

The removal of AALLSAMEBPV default constructor broke the StrippingCache. The solution is simply to switch any instance of 'AALLSAMEBPV' to 'AALLSAMEBPV(-1, -1, -1)' to restore the old behaviour. This is done in the StrippingCache options.

Added 1 commit:

• b99d56f1 - Fixing options for StrippingCache

Compare with previous version

Hi @malexand , you have opened the MR !42 (merged) but there is nothing to be merged in there, since everything has already been committed here. Is that what you wanted to do? I will just close this MR, then.

Mentioned in commit b99d56f1

Note that there might be a real problem in some cases by using AALLSAMEBPV(-1, -1, -1). I still don't know why, but according to @ibelyaev this leads to undefined behaviour.

The behaviour is not "undefined", it is just wrong...

Yeah, @erodrigu I'd forgotten to do 'git add' the first time ;-)

@rmatev, the StrippingCache options are currently just for legacy strippings, so I've kept things consistent with how they were actually run by using 'AALLSAMEBPV(-1, -1, -1)'. We'll have to define some sensible values for the arguments for the new strippings.

Mentioned in commit 57830159

Status changed to merged