@hyperion, you're right, commits such as 15713 or 19095 ought to be avoided. On the other hand, committing every single modification separately is undesirable too. Where to draw the line is always a bit arbitrary. D2493, D2494, D2495 affect the same entities; I did it in three because each of those can make sense on its own (or not) and can be implemented indepently.
D2494 removes splash attacks and raises default damage of artillery, but having one patch removing splash attacks and another raising default damage is not really an improvement, these changes make much more sense together than they do separately. Likewise, the <PreferredClasses> changes are because of the damage changes. You have a point that the summary does not really explain why the patch is as it is. I'll rewrite the summary to be more argumentative rather than descriptive.
One problem is that the actual splash damage inflicted depends not just on the splash radius, but also on the distance from the centre of the entity hit; because structures (and warships) tend to have long footprints (longer than the splash radius), they don't take any splash damage in practice. Also, the AI does not seem able to take splash damage into account. Another issue is that ancient missiles were not exploding shells; splash damage is anachronistic.
To be clear, I'm certainly not arguing to remove the mechanic. I'm actually in favour of having a cheat unit that has a splash attack (perhaps a mortar; or some Greek fire, or the dragon?).
Anyway, this discussion is getting quite specific. Could we please continue it over there on phabricator ( https://code.wildfiregames.com/D2494 ), where the actual review is supposed to take place?