[Github-comments] [geany/geany] WIP: Windows (and Linux) spawn fixes (#1300)

[Matthew Brush]

codebrainz commented on this pull request.



> +} Placeholder;
+
+
+static const gchar *placeholder_replacement(Placeholder *placeholders, size_t n_placeholders, gchar placeholder)
+{
+	for (size_t i = 0; i < n_placeholders; i++)
+	{
+		if (placeholders[i].placeholder == placeholder)
+			return placeholders[i].replacement;
+	}
+
+	return NULL;
+}
+
+
+/* Replaces %s-style placeholders in a string.

Just out of curiosity, why not using template-style placeholders and re-using the template replacement code?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1300#pullrequestreview-8303608
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20161112/672da395/attachment.html>