NSAtomicStore is added! #421

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants
@ilin-in

ilin-in commented Feb 20, 2013

Now you can setup Core Data stack with manual Store Type name. That
means you can use MagicalRecord with NSAtomicStore easily.

NSAtomicStore added
Now you can setup Core Data stack with manualy Store Type name. That
means you can use MagicalRecord with NSAtomicStore easily.
@casademora

This comment has been minimized.

Show comment Hide comment
@casademora

casademora Feb 20, 2013

Let's not make a dictionary of parameters :( This is not very discoverable.

Let's not make a dictionary of parameters :( This is not very discoverable.

This comment has been minimized.

Show comment Hide comment
@ilin-in

ilin-in Feb 20, 2013

Owner

ok. What is better to use here define new class or struct? performSelector:withObject:afterDelay: doesn't accept 2 params :(

Owner

ilin-in replied Feb 20, 2013

ok. What is better to use here define new class or struct? performSelector:withObject:afterDelay: doesn't accept 2 params :(

This comment has been minimized.

Show comment Hide comment
@ilin-in

ilin-in Feb 21, 2013

Owner

I'm have fixed this issue in b30bb2b by replacing performSelector:withObject:afterDelay: to dispatch_after

Owner

ilin-in replied Feb 21, 2013

I'm have fixed this issue in b30bb2b by replacing performSelector:withObject:afterDelay: to dispatch_after

Array params have been removed
`performSelector:withObject:afterDelay` replaced to `dispatch_after` in
same time
@tonyarnold

This comment has been minimized.

Show comment Hide comment
@tonyarnold

tonyarnold Feb 25, 2013

Contributor

@ilin-in, would you mind resubmitting this pull request against the develop branch? We try not to merge changes directly to the stable master branch if we can avoid it.

Contributor

tonyarnold commented Feb 25, 2013

@ilin-in, would you mind resubmitting this pull request against the develop branch? We try not to merge changes directly to the stable master branch if we can avoid it.

@ilin-in

This comment has been minimized.

Show comment Hide comment
@ilin-in

ilin-in Feb 25, 2013

@tonyarnold yes, sure.

ilin-in commented Feb 25, 2013

@tonyarnold yes, sure.

@ghost ghost assigned tonyarnold Apr 21, 2013

@tonyarnold

This comment has been minimized.

Show comment Hide comment
@tonyarnold

tonyarnold Apr 21, 2013

Contributor

Closing in favour of #423.

Contributor

tonyarnold commented Apr 21, 2013

Closing in favour of #423.

@tonyarnold tonyarnold closed this Apr 21, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment