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

Felix Paul Kühne fkuehne at videolan.org
Sat Jul 12 12:21:06 CEST 2014

On 12.07.2014, at 12:19, Florent Pillet <fpillet at gmail.com> wrote:

> 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.

ehm, yes, this is absolutely a good idea :)



More information about the vlc-devel mailing list