Can we have some more expressive names. Names such as "data" should be "keychain_item" and "_load_item_data" could be "_load_keychain_item" or just "_load_item" since we're already in the keychain context.