-
Notifications
You must be signed in to change notification settings - Fork 45
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
Observer can now be extended #34
Conversation
@@ -133,3 +133,4 @@ solving of bugs, etc.) from the following developers : | |||
* Koert Gaaikema (@koertgaaikema) | |||
* Matt Logan (@mattlogan) | |||
* Aran Arunakiri | |||
* @rckrdstrgrd |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Credit where credit is due :)
void handleNotification( int aNotificationType ); | ||
void handleNotification( int aNotificationType, int aValue ); | ||
virtual void handleNotification( int aNotificationType ); | ||
virtual void handleNotification( int aNotificationType, int aValue ); | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functions can now be overridden by inheriting classes.
delete aEvent; | ||
aEvent = 0; | ||
delete audioEvent; | ||
audioEvent = 0; | ||
#endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you're going to clear memory, do it on properties that actually exist, yo.
Ensuring Observer can be extended as an C++ class, also fixing some bugs by observation (punny). Both cases noted by @rckrdstrgrd