Robert Shearman rob@codeweavers.com writes:
Why did you decide to rely on another function to map the access rights? It seems that none of the objects' mappings depend on any state in the object, so couldn't you have just put the access rights that GENERIC_READ, GENERIC_WRITE, GENERIC_EXECUTE and GENERIC_ALL map to somewhere associated with the object and have just one mapping function.
I think it's cleaner to use functions in the ops structure rather than starting to mix various data pointers in there too.