Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

changed OFGenerateUUIDString to take void arguments for ios compilation #31

Merged
merged 3 commits into from Apr 23, 2012

Conversation

matthewchung
Copy link

Using XCode 4.3.1 (LLVM 3.1) for ios was generating warnings as noted below. passing void as the argument removed that warning.

Submodules/ShareKit/Classes/ShareKit/Sharers/Services/Flickr/../../../../../Submodules/objectiveflickr/Source/OFUtilities.m:68:11: warning: no previous prototype for function 'OFGenerateUUIDString' [-Wmissing-prototypes]
NSString *OFGenerateUUIDString()
^
1 warning generated.

@langk
Copy link

langk commented Mar 19, 2012

Cool - just wanted to add the same pull request, but both for the .h and .m file. Would appreciate if this is pulled.

@matthewchung
Copy link
Author

just made changes for both the .h and .m files.

---Matthew Chung

Date: Mon, 19 Mar 2012 14:57:09 -0700
From: reply@reply.github.com
To: matthew_chung@hotmail.com
Subject: Re: [objectiveflickr] changed OFGenerateUUIDString to take void arguments for ios compilation (#31)

Cool - just wanted to add the same pull request, but both for the .h and .m file. Would appreciate if this is pulled.


Reply to this email directly or view it on GitHub:
#31 (comment)

@cleishm
Copy link

cleishm commented Apr 19, 2012

+1

lukhnos added a commit that referenced this pull request Apr 23, 2012
Change OFGenerateUUIDString to take void arguments for iOS compilation.
@lukhnos lukhnos merged commit e6df2db into lukhnos:master Apr 23, 2012
@lukhnos
Copy link
Owner

lukhnos commented Apr 23, 2012

Thanks for the pull request!

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.

None yet

4 participants