Skip to content

Update ExifContainer.m#10

Open
meet881994 wants to merge 1 commit intoNikita2k:masterfrom
meet881994:patch-1
Open

Update ExifContainer.m#10
meet881994 wants to merge 1 commit intoNikita2k:masterfrom
meet881994:patch-1

Conversation

@meet881994
Copy link
Copy Markdown

Line 86 -
[self setValue:comment forExifKey:key];

This passes value as "comment" and Exifkey as "Key"

It calls following function

Line 110 -

  • (void)setValue:(NSString *)key forExifKey:(NSString *)value {
    [self.exifDictionary setObject:value forKey:key];
    }

  • Issue : In line 86 we have value as first parameter and key as second parameter , this must be same in line 110 but it is "reversed" so corrected function

  • (void)setValue:(NSString *)value forExifKey:(NSString *)key {
    [self.exifDictionary setObject:value forKey:key];
    }

 Line 86 - 
[self setValue:comment forExifKey:key];

This passes value as "comment" and Exifkey as "Key"


It calls following function 

Line 110 -
- (void)setValue:(NSString *)key forExifKey:(NSString *)value {
    [self.exifDictionary setObject:value forKey:key];
}


- Issue : In line 86 we have value as first parameter and key as second parameter , this must be same in line 110 but it is "reversed" so corrected function

- (void)setValue:(NSString *)value forExifKey:(NSString *)key {
    [self.exifDictionary setObject:value forKey:key];
}
@cugxy
Copy link
Copy Markdown

cugxy commented Jul 24, 2019

I found this bug too。

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants