IniFile: Added common code to migrate ports.

Old settings have IPv4 and IPv6 ports as separate values, new settings have a single value for the ports.

src/IniFile.cpp

@@ -888,3 +888,39 @@ void cIniFile::RemoveBom(AString & a_line) const
 
 
 
+
+AStringVector ReadUpgradeIniPorts(
+	cIniFile & a_IniFile,
+	const AString & a_KeyName,
+	const AString & a_PortsValueName,
+	const AString & a_OldIPv4ValueName,
+	const AString & a_OldIPv6ValueName,
+	const AString & a_DefaultValue
+)
+{
+	// Read the regular value, but don't use the default (in order to detect missing value for upgrade):
+	AStringVector Ports = StringSplitAndTrim(a_IniFile.GetValue(a_KeyName, a_PortsValueName), ";,");
+
+	if (Ports.empty())
+	{
+		// Historically there were two separate entries for IPv4 and IPv6, merge them and migrate:
+		AString Ports4 = a_IniFile.GetValue(a_KeyName, a_OldIPv4ValueName, a_DefaultValue);
+		AString Ports6 = a_IniFile.GetValue(a_KeyName, a_OldIPv6ValueName);
+		Ports = MergeStringVectors(StringSplitAndTrim(Ports4, ";,"), StringSplitAndTrim(Ports6, ";,"));
+		a_IniFile.DeleteValue(a_KeyName, a_OldIPv4ValueName);
+		a_IniFile.DeleteValue(a_KeyName, a_OldIPv6ValueName);
+
+		// If those weren't present or were empty, use the default:"
+		if (Ports.empty())
+		{
+			Ports = StringSplitAndTrim(a_DefaultValue, ";,");
+		}
+		a_IniFile.SetValue(a_KeyName, a_PortsValueName, StringsConcat(Ports, ','));
+	}
+
+	return Ports;
+}
+
+
+
+

src/IniFile.h

@@ -15,9 +15,7 @@
 !! MODIFIED BY FAKETRUTH and madmaxoft!!
 */
 
-#ifndef CIniFile_H
+#pragma once
-#define CIniFile_H
-
 
 
 
@@ -215,4 +213,22 @@ public:
 
 // tolua_end
 
-#endif
+
+
+
+
+/** Reads the list of ports from the INI file, possibly upgrading from IPv4/IPv6-specific values into new version-agnostic value.
+Reads the list of ports from a_PortsValueName. If that value doesn't exist or is empty, the list is combined from values
+in a_OldIPv4ValueName and a_OldIPv6ValueName; in this case the old values are removed from the INI file.
+If there is none of the three values or they are all empty, the default is used and stored in the Ports value. */
+AStringVector ReadUpgradeIniPorts(
+	cIniFile & a_IniFile,
+	const AString & a_KeyName,
+	const AString & a_PortsValueName,
+	const AString & a_OldIPv4ValueName,
+	const AString & a_OldIPv6ValueName,
+	const AString & a_DefaultValue
+);
+
+
+