Open DartRuffian opened 1 day ago
I think these changes could break things and just using
QEGVAR(common,field_rations_blockUpdates)
is not that bad
True, but it makes weird naming conventions. It also doesn't make a lot of sense to someone reading through the code until they get to addType
and see that the event gets ace_common_
added to the name.
When merged this pull request will:
"blockDamage"
->QGVAR(blockDamage)
ace_common_
appended to it, see #10462IMPORTANT
Component - Add|Fix|Improve|Change|Make|Remove {changes}
.