Redundant condition related to LFNST

[T.Hashimoto]

In 8.7.4.1 of JVET-R2001-v10.doc:

The variables nonZeroW and nonZeroH are derived as follows:
–	If ApplyLfnstFlag is equal to 1 and nTbW is greater than or equal to 4 and nTbH is greater than or equal to 4, the following applies:

Above size restriction part "and both nTbW and nTbH are greater than or equal to 4" is redundant since ( lfnst_idx > 0 && cIdx = = 0 ) check in ApplyLfnstFlag already covered the case.
Thus I'd like to suggest to remove "and both nTbW and nTbH are greater than or equal to 4" as follows

The variables nonZeroW and nonZeroH are derived as follows:
–	If ApplyLfnstFlag is equal to 1, the following applies:

[bbross]

Good catch, I assume the check in the same section above is redundant as well?

When ApplyLfnstFlag is equal to 1, transform_skip_flag[ xTbY ][ yTbY ][ cIdx ] is equal to 0 and both nTbW and nTbH are greater than or equal to 4, the following applies:
– The variables predModeIntra, nLfnstOutSize, log2LfnstSize, nLfnstSize, and nonZeroSize are derived as follows:
...

[T.Hashimoto]

Yes, I agree. It should be redundant.
A suggested fix:

When ApplyLfnstFlag is equal to 1 and transform_skip_flag[ xTbY ][ yTbY ][ cIdx ] is equal to 0, the following applies:
– The variables predModeIntra, nLfnstOutSize, log2LfnstSize, nLfnstSize, and nonZeroSize are derived as follows:

[bbross]

Will be fixed in JVET-S2001-vA.