Generic cache class about ev3dev-lang-python HOT 8 CLOSED

This is a file handle cache - the purpose of the extra bits in the device version is that it allows you to open a file with only the mode you ask for - maybe we can modify things so that if an attribute is r/w you get the right mode the first time :-) The reopen is only needed if you write to an attribute, and then read it later.

from ev3dev-lang-python.

We can also make the cache key to be file_name/mode. This will have two versions of the same file in cache, but would not require reopen.

from ev3dev-lang-python.

I'm not sure having one descriptor open for read and another for write is a good thing - I guess as long as the seek points are truly independent it will work - not sure though.

from ev3dev-lang-python.

It certainly works in C++ bindings. I agree it sounds a bit wasteful. And the trick with reopening on exception should work for all cases.

from ev3dev-lang-python.

Although it's not clear how to incorporate the trick into cache, since the try block should be around both cache access and file operation. May be we should think not about file handle cache, but file reader /writer?

from ev3dev-lang-python.

+1 File reader/writer (that takes advantage of a cache for file handles) :-)

from ev3dev-lang-python.

Something like this: https://github.com/ddemidov/ev3dev-lang-python-1/commit/00ed2429497828589da087ce0ba3cea2b6357348?

from ev3dev-lang-python.

Resolved by #31

from ev3dev-lang-python.