Replace traditional SbxParams::const_iterator loop with modern range-based for loop in SbxVariable::GetName() for better readability and modern C++ style while preserving the comma-separation logic between parameters. Change-Id: Ib978b29b70bf6a0a85bf94be1a3dd67fea45529d Reviewed-on: https://gerrit.libreoffice.org/c/core/+/193379 Tested-by: Jenkins Reviewed-by: Hossein <hossein@libreoffice.org>