[vlc-devel] [PATCH 1/9] VLCKit: migrate VLCMedia to ARC

Florent Pillet fpillet at gmail.com
Sat Jul 12 12:19:22 CEST 2014

On 12 Jul 2014, at 11:46, Felix Paul Kühne <fkuehne at videolan.org> wrote:

> Hey Florent,
>> - at property (readonly) NSDictionary * metaDictionary;
>> + at property (nonatomic, readonly, strong) NSDictionary * metaDictionary;
> Please follow Caro's advise and add the copy attribute.
> Rest of the patch LGTM.

Ok, sure. Then do you think the iOS version of the -(NSDictionary *)metaDictionary getter should return a copy too? Because in iOS case you are always returning the original mutable dictionary, regardless of the property declared attributes.


More information about the vlc-devel mailing list