-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
[video] use action listener pattern for CPlayerController #7581
[video] use action listener pattern for CPlayerController #7581
Conversation
There's german in your commit message. |
0f8f8a0
to
ae0960c
Compare
@Montellese thanks, rebased. |
CPlayerController(); | ||
CPlayerController(const CPlayerController&); | ||
CPlayerController& operator=(CPlayerController const&); | ||
virtual ~CPlayerController(); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Add the virtual that @Memphiz mentioned and it's good to go imo |
ae0960c
to
6f0d5cd
Compare
@Memphiz done jenkins build this please |
build erros are unrelated |
…ener [video] use action listener pattern for CPlayerController
As the title said this refactors
CPlayerController
to use the action listener interface/pattern and register/unregister itCApplication
. It also changeCPlayerController
to be Singleton so we can drop the member fromCApplication
.@Paxxi mind taking a look