yes as leo says i think this should be OF_KEY_LEFT_SHIFT but apart from that i think it looks good