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

Fix annotation for xdg_mime_list_parents (SL #4530) #27

Closed
wants to merge 1 commit into from

Conversation

walterbender
Copy link
Member

As per SL #4530, there is a fatal crash in Sugar when retrieving mimetypes for some Journal entries when contructing the icon palette (for both the Journal list view and detail view). We isolated the problem to xdgmime.c, the call to xdg_get_mime_parents. Dnarvaez observed that the annotation for the return argument was missing and that the annotation for the return for mime_list_parents was wrong. This patch fixes these annotations.

As per SL #4530, there is a fatal crash in Sugar when retrieving mimetypes for some Journal entries when contructing the icon palette (for both the Journal list view and detail view). We isolated the problem to xdgmime.c, the call to xdg_get_mime_parents. Dnarvaez observed that the annotation for the return argument was missing and that the annotation for the return for mime_list_parents was wrong. This patch fixes these annotations.
@tchx84
Copy link
Member

tchx84 commented Aug 15, 2013

Just tested (compared) it and it works!

@dnarvaez
Copy link
Contributor

I guess this unintentional, but better avoid creating feature branches on the official repo.

@dnarvaez dnarvaez closed this Aug 15, 2013
@dnarvaez
Copy link
Contributor

I deleted the branch. Also I modified the commit log to not exceed 80 lines and to say "Fixes #4530" which triggers the trac hook and closes the ticket automatically

@walterbender
Copy link
Member Author

Yes. I pushed to the wrong repo.

-walter

On Thu, Aug 15, 2013 at 5:54 PM, Daniel Narvaez notifications@github.comwrote:

I guess this unintentional, but better avoid creating feature branches on
the official repo.


Reply to this email directly or view it on GitHubhttps://github.com//pull/27#issuecomment-22733668
.

Walter Bender
Sugar Labs
http://www.sugarlabs.org

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

3 participants