Build on Windows Machine

[utap]

Build on Windows Machine

[hadisfr]

Hi!
Unfortunately, we do not support Windows and MinGW officially.
It seems that we have used non-POSIX gcc-specific old-C-style getline there, which is not available in MinGW (See https://stackoverflow.com/q/27381903).
If you have found any workaround, merge-requests are appreciated. :D

Do you have any solution? 🤔 @gsoosk @sadmanbrad

[AmRo045]

I think we can change the implementation of readMapFromFile method from this:

APHTTP/utils/utilities.cpp

Lines 209 to 246 in 03a8239

	int readMapFromFile(std::string fname, std::map<std::string, std::string> *m) {
	int count = 0;
	FILE *fp = fopen(fname.c_str(), "r");
	if (!fp)
	return -errno;
	m->clear();
	char *buf = 0;
	size_t buflen = 0;
	while (getline(&buf, &buflen, fp) > 0) {
	char *nl = strchr(buf, '\n');
	if (nl == NULL)
	continue;
	*nl = 0;
	char *sep = strchr(buf, '=');
	if (sep == NULL)
	continue;
	*sep = 0;
	sep++;
	std::string s1 = buf;
	std::string s2 = sep;
	(*m)[s1] = s2;
	count++;
	}
	if (buf)
	free(buf);
	fclose(fp);
	return count;
	}

to this:

int readMapFromFile(std::string fname, std::map<std::string, std::string> *m) {
  ifstream inputStream(fname);
  std::string line;

  if (inputStream.fail())
    return -errno;

  while (getline(inputStream, line)) {
    auto tokens = tokenize(line, '=');
    //     KEY         VALUE
    (*m)[tokens[0]] = tokens[1];
  }

  return (*m).size();
}

[hadisfr]

Thank you. Your code seems much cleaner than the old one. :D I did not remember internals of library, so I'll ask someone else to review it and apply the patch if everything is OK. A MR will be appreciated, too. :D