Skip to content

Loading…

Fix what might be an overlooked bug in method. #406

Closed
wants to merge 1 commit into from

2 participants

@berkus

I wonder why XCode doesn't complain here.

@berkus berkus Fix what might be an overlooked bug in method.
I wonder why XCode doesn't complain here.
25cebe2
@casademora
Magical Panda Software member

This is not a bug. The ; at the end of a method definition is perfectly valid Objective C. We use this 1) out of habit, and 2) it becomes easier to copy/paste and manage the public header file with this at the end of the line.

Alright then, sorry for the noise. I'm only new in objc land, and MagicalRecord seriously rocks - I just threw out 10kb of boilerplate cruft. Keep it up!

@berkus berkus closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Feb 4, 2013
  1. @berkus

    Fix what might be an overlooked bug in method.

    berkus committed
    I wonder why XCode doesn't complain here.
Showing with 1 addition and 1 deletion.
  1. +1 −1 MagicalRecord/Categories/NSManagedObject/NSManagedObject+MagicalFinders.m
View
2 MagicalRecord/Categories/NSManagedObject/NSManagedObject+MagicalFinders.m
@@ -231,7 +231,7 @@ + (NSFetchedResultsController *) MR_fetchController:(NSFetchRequest *)request de
return controller;
}
-+ (NSFetchedResultsController *) MR_fetchAllWithDelegate:(id<NSFetchedResultsControllerDelegate>)delegate;
++ (NSFetchedResultsController *) MR_fetchAllWithDelegate:(id<NSFetchedResultsControllerDelegate>)delegate
{
return [self MR_fetchAllWithDelegate:delegate inContext:[NSManagedObjectContext MR_contextForCurrentThread]];
}
Something went wrong with that request. Please try again.