My patch #44930 suffers from some A_NONE / A_NEVER confusion. A_NONE is 0, and A_NEVER is NULL which is also 0.
I'd change value of A_NONE to 1, so it would be different from A_NEVER, and more importantly, compiler would give an error when one tries to put that value to a pointer. Negligible con would be that one slot in the beginning of various arrays would get wasted.
The other A_xxx values need to be similarly increased by one.
My patch #44930 suffers from some A_NONE / A_NEVER confusion. A_NONE is 0, and A_NEVER is NULL which is also 0.
I'd change value of A_NONE to 1, so it would be different from A_NEVER, and more importantly, compiler would give an error when one tries to put that value to a pointer. Negligible con would be that one slot in the beginning of various arrays would get wasted.
The other A_xxx values need to be similarly increased by one.