search

GrandOrgue / grandorgue

GrandOrgue software
Other
172 stars 43 forks source link

Enchanced code style of GOEventHandlerList #1925

Closed oleg68 closed 4 months ago

oleg68 commented 5 months ago

This is the first PR related to #1816

Earlier GOEventHandlerList.cpp contained a lot of copy-pasted code, so adding a new method requires to copy-paste more.

This PR reorganises this class so all the reusable code is in the methods of the new inner class UPVector. Now most of methods only forward to a corresponding method of UPVector.

It is just refactoring. No GO behavior should be changed.

oleg68 commented 4 months ago

@rousseldenis could you approve this PR?