Good point

> On Oct 5, 2013, at 5:49 AM, Justin Spahr-Summers <notifications@github.com> wrote:
> 
> In Mantle/NSValueTransformer+MTLPredefinedTransformerAdditions.m:
> 
> > @@ -49,14 +49,14 @@ + (NSValueTransformer *)mtl_JSONDictionaryTransformerWithModelClass:(Class)model
> >  	NSParameterAssert([modelClass conformsToProtocol:@protocol(MTLJSONSerializing)]);
> >  
> >  	return [MTLValueTransformer
> > -		reversibleTransformerWithForwardBlock:^ id (id JSONDictionary) {
> > +		reversibleTransformerWithForwardTransformation:^ id (id JSONDictionary, NSError **error) {
> >  			if (JSONDictionary == nil) return nil;
> I would follow RAC's -firstOrDefault:success:error: here:
> 
> typedef id (^MTLValueTransformationBlock)(id, BOOL *, NSError **);
> Pointers to primitives are easier to reason about than pointers to objects.
> 
> —
> Reply to this email directly or view it on GitHub.