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

Already on GitHub? Sign in to your account

removed bAsync param from Observable::NotifyObservers() #1678

Merged
merged 1 commit into from Oct 26, 2012

Conversation

Projects
None yet
1 participant
Member

opdenkamp commented Oct 26, 2012

the announcementmanager hack that disables this when xbmc shuts down (so it doesn't try to use deleted stuff) could lead to deadlocks if an observable is created by something in Announce(), and it should be done properly if and when we need this instead

fixed - removed bAsync param from Observable::NotifyObservers(). the …
…announcementmanager hack that disables this when xbmc shuts down (so it doesn't try to use deleted stuff) could lead to deadlocks if an observable is created by something in Announce(), and it should be done properly if and when we need this instead

opdenkamp pushed a commit that referenced this pull request Oct 26, 2012

Merge pull request #1678 from opdenkamp/noasyncobs
removed bAsync param from Observable::NotifyObservers()

@opdenkamp opdenkamp merged commit db72690 into xbmc:master Oct 26, 2012

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