api: IOProxy const method adjustments

AcademySoftwareFoundation / OpenImageIO

Reading, writing, and processing images in a wide variety of file formats, using a format-agnostic API, aimed at VFX applications.

https://openimageio.readthedocs.org

Apache License 2.0

1.98k stars 597 forks source link

Closed lgritz closed 2 months ago

lgritz commented 2 months ago

With 3.0 approaching, it seems like the only time to make these changes that I made note of a while back:

It seems to me that tell() should be const because it isn't going to modify or change the state of the IOProxy.
It seems to me that flush() should NOT be const, because it probably does change internal state.

lgritz commented 2 months ago

Does this seem right to you? Is my intuition correct about changing these two methods?