[Github-comments] [geany/geany] Improve goto-symbols popup (#1445)

Colomban Wendling notifications at xxxxx
Sat Nov 24 17:39:17 UTC 2018


b4n requested changes on this pull request.

Still annoying :)

Otherwise looks good.

You also should resolve the conflict, but apart from that I think we're good do go.

> @@ -2031,6 +2031,208 @@ gchar **utils_strv_join(gchar **first, gchar **second)
 	return strv;
 }
 
+/* * Returns the common prefix in a list of strings.
+ *
+ * The size of the list may be given explicitely, but defaults to @c g_strv_length(strv).
+ *
+ * @param strv The list of strings to process.
+ * @param num The number of strings contained in @a strv. Can be -1 if it's terminated by @c NULL.
+ *
+ * @return The common prefix that is part of all strings (maybe empty), or NULL if an empty list
+ *         was passed in.
+ */
+static gchar *utils_strv_find_common_prefix(gchar **strv, gssize num)
+{
+	gchar *prefix;

unused now

-- 
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/1445#pullrequestreview-178057077
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20181124/8cd48bd1/attachment-0001.html>


More information about the Github-comments mailing list