So maybe add something like ASSERT(features->update_parameter_hash()) or let it be like it is (i mean without this strange line)?
I prefer without this line, since i didn't find the good reason why it should be there.
And you?