feat: make file-related APIs compatible with io::Cursor

[davidepedranz]

The codebase defines a StorageFile trait to model anything similar to a file. By using impl StorageFile instead of fs::File in the API signatures, users are able to provide either a file or something similar (for example, io::Cursor to manipulate a block of data in memory).

This commit updates the APIs still using fs::File to impl StorageFile.

[davidepedranz]

A few tests (tag::tests::aiff_read_and_write and tag::tests::test_issue_39) are failing, but they were failing on both the latest commit on main and the latest commit in v1.10.0. They are 99% unrelated to this change. Also, this change should break compilation if anything, not really change the code behaviour.

[davidepedranz]

Thanks a lot for creating this library: I found it very easy and convenient to use!

If you are open to contributions, I have a few additional PR to open to add support for some other v2.4 frames (TPOS, TDRL) 🙏🏼

[davidepedranz]

^ reformatted the file, for some reason cargo fmt didn't run automatically in my IDE

[polyfloyd]

Thanks! Will have a look later this week :)

[polyfloyd]

Nice work!

And also, I am always happy to receive contributions to implement support for more frame types :)

[davidepedranz]

Thanks for the quick review and merging this change, @polyfloyd!

And also, I am always happy to receive contributions to implement support for more frame types :)

Amazing! I will work on adapting the code (currently, I am extending your library with external functions) and submit a few other PRs in the next few days.