The commits here don't follow the guidelines: https://wiki.winehq.org/Submitting_Patches#Patch_guidelines
The "is_settable" code is added and later removed, and `find_value_pos` is introduced broken and later fixed.
I think it'd be OK to squash these into one commit.