Hi, thanks for your contribution!
Looks fine to me. The saving function itself already sorts everything else, so this should be the last point where it can be indeterministic. While at it, could you update the foreach to a for? Not all code already agrees with our style guide, but we try to update it whenever we are touching it [0]. If you haven't done so already, you need to agree to the Harmony CLI before we can accept your contribution [1]. Gave it a spin and ran some configurations, so you can consider this: Tested-by: Folke Gleumes <f.gleu...@proxmox.com> [0] https://pve.proxmox.com/wiki/Perl_Style_Guide [1] https://pve.proxmox.com/wiki/Developer_Documentation#Software_License_and_Copyright On Tue, 2024-04-16 at 09:27 +0200, Daniel Krambrock via pve-devel wrote: > _______________________________________________ > pve-devel mailing list > pve-devel@lists.proxmox.com > https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel