Closed jamesmblair closed 8 years ago
Awesome work James! This looks great. Using the struct for the options values was great call, helps keep the method signatures down. I'm going to merge this in and continue the discussion here https://github.com/jrendel/SwiftKeychainWrapper/issues/38.
This is a sample implementation of the method parameter options design I had in mind for SecClass and SecAttrAccessible. The options type could be could be extended further for other attributes. I also added method overloads in the KeychainWrapper class to support the options parameter. What are your thoughts on an implementation like this?