b4n requested changes on this pull request.

Otherwise looks good

>   *
- *  @param msg_color A color to be used for the text. It must be an element of 
#MsgColors.
- *  @param format @c printf()-style format string.
- *  @param ... Arguments for the @c format string.
+ * @see msgwin_compiler_add_string()
+ *
+ * @since 0.15

Is that correct?  Looks to me it's new in 
921f010ffc38aaadf7118161b2f85df58e7bfc5c which seems to be 
[0.16](https://github.com/geany/geany/blob/921f010ffc38aaadf7118161b2f85df58e7bfc5c/configure.in#L5)
It doesn't matter much at all, but wrong info isn't nice.

> @@ -488,6 +519,32 @@ void msgwin_status_add(const gchar *format, ...)
        }
 }
 
+/**
+ * Logs a formatted status message *without* setting the status bar.
+ *
+ * Use @ref ui_set_statusbar() to display text on the statusbar.
+ *
+ * @param format @c printf()-style format string.
+ * @param ...    Arguments for the @c format string.
+ *
+ * @see msgwin_status_add_string()
+ *
+ * @since 0.15

Same

-- 
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/1748#pullrequestreview-91774332

Reply via email to