kam_remake
kam_remake copied to clipboard
reyandme
fix compilation with FPC + Lazarus of campaigns sorting algorithm
FPC and Lazarus on Linux do not have List<T>.List[] property, but List<T> fortunately has Sort() method at Linux platform with a comparator to give from a call site
Please, consider these changes to merge them in. The previous business logic of Windows's implementation has been preserved
I think these changes are easy to verify. We need to get list of current campaigns in the game and see that they are sorted according to the rules. Then we could create a dummy campaign, if it is possible, with the smallest efforts undertaken and name them with TSK, TPR and different other names and see that they are again sorted. Or, if it is possible and there is a list of such campaigns in resources, we might create dummy files and see that their names are sorted according to the rules
Yes, this or similar test should be done. Could you do such a test please ? I would be able to do it, but some time later.
Actually, I don't have Windows, neither installed nor an installer to install
as in the other MR, I could test those changes eventually it seems like a low risk change
as in the other MR, I could test those changes eventually it seems like a low risk change
I would appreciate if you did