Could you put this functionality in a `self.read_null()` method instead of having it happen in `ion_type()`?
There are three other raw value types besides LazyRawBinaryValue_1_1:
LazyRawBinaryValue_1_0
LazyRawTextValue_1_1
LazyRawTextValue_1_0
Each of them has an ion_type() method and I'm pretty sure at least some of them return IonType::Null if the value is a null value, even if it's a typed null. The behavior you've implemented is probably more correct, but if we make this change here we'd need to make it in the other three places too.
There are three other raw value types besides
LazyRawBinaryValue_1_1:LazyRawBinaryValue_1_0LazyRawTextValue_1_1LazyRawTextValue_1_0Each of them has an
ion_type()method and I'm pretty sure at least some of them returnIonType::Nullif the value is a null value, even if it's a typed null. The behavior you've implemented is probably more correct, but if we make this change here we'd need to make it in the other three places too._Originally posted by @zslayton in https://github.com/amazon-ion/ion-rust/pull/826#discussion_r1742481341_