Redundant is_metadata_valid_for invocation

[drucker-nir]

Hello,

I believe that the 6 is_valid_for functions (L280, L295, L310, L325, L340, L355) invoke the is_metadata_valid_for functions twice. The first invocation is always a direct invocation and the second is through the is_data_valid_for functions (e.g., in L239,L287).
It is not a bug, but it complicates the code and can add a (probably low) performance penalty.

Thanks,
Nir Drucker

[WeiDaiWD]

That is true! They are removed for the next release. Also L379 has the same problem. Thank you so much for catching this.