<p><b>@b4n</b> requested changes on this pull request.</p>

<p>Looks mostly OK, but I'd like the few small fixes</p><hr>

<p>In <a href="https://github.com/geany/geany/pull/1748#discussion_r163651576">src/msgwindow.c</a>:</p>
<pre style='color:#555'>> @@ -488,6 +508,30 @@ void msgwin_status_add(const gchar *format, ...)
        }
 }
 
+/**
+ * Logs a 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.
+ **/
+GEANY_API_SYMBOL
+void msgwin_status_add(const gchar *format, ...)
+{
+       GtkTreeIter iter;
</pre>
<p>unused variable</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/1748#discussion_r163651595">src/msgwindow.c</a>:</p>
<pre style='color:#555'>> @@ -488,6 +508,30 @@ void msgwin_status_add(const gchar *format, ...)
        }
 }
 
+/**
+ * Logs a 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.
+ **/
+GEANY_API_SYMBOL
+void msgwin_status_add(const gchar *format, ...)
+{
+       GtkTreeIter iter;
+       gchar *string;
+       gchar *statusmsg, *time_str;
</pre>
<p>unused variables</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/1748#discussion_r163652024">src/msgwindow.c</a>:</p>
<pre style='color:#555'>> @@ -410,7 +423,19 @@ void msgwin_msg_add(gint msg_color, gint line, GeanyDocument *doc, const gchar *
 }
 
 
-/* adds string to the msg treeview */
+/**
+ * Adds a new message in the messages tab treeview in the messages window.
+ *
+ * If @a line and @a doc are set, clicking on this line jumps into the
+ * file which is specified by @a doc into the line specified with @a line.
+ *
+ * @param msg_color A color to be used for the text. It must be an element of #MsgColors.
+ * @param line      The document's line where the message belongs to. Set to @c -1 to ignore.
+ * @param doc       @nullable The document. Set to @c NULL to ignore.
+ * @param string    Message to be added.
+ *
+ * @since @todo
+ **/
 void msgwin_msg_add_string(gint msg_color, gint line, GeanyDocument *doc, const gchar *string)
</pre>
<p>Needs a <code>GEANY_API_SYMBOL</code> marker, doesn't it?</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/1748#discussion_r163652586">src/msgwindow.h</a>:</p>
<pre style='color:#555'>>  
 void msgwin_compiler_add(gint msg_color, const gchar *format, ...) G_GNUC_PRINTF (2, 3);
+void msgwin_compiler_add_string(gint msg_color, const gchar *msg);
</pre>
<p>you should remove the duplicated prototype from the <code>GEANY_PRIVATE</code> section below</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/1748#discussion_r163652920">src/msgwindow.h</a>:</p>
<pre style='color:#555'>>  
 void msgwin_msg_add(gint msg_color, gint line, GeanyDocument *doc, const gchar *format, ...)
                        G_GNUC_PRINTF (4, 5);
+void msgwin_msg_add_string(gint msg_color, gint line, GeanyDocument *doc, const char *msg);
</pre>
<p>Ditto for that one that already existed</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/1748#discussion_r163653512">src/msgwindow.c</a>:</p>
<pre style='color:#555'>>   *
- *  @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.
+ * @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.
+ *
</pre>
<p>Could be interesting to add a <code>@see msgwin_compiler_add_string()</code>, wouldn't it?  And similar for the other functions.  Just a suggestion though, but could avoid a little confusion for the C API users.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/1748#pullrequestreview-91299682">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ0koEtiICsBzA1i8A39RSdrQcXBpks5tN4VfgaJpZM4RqazS">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ5YF9SMIV8f4BNF2GmPJ1noaTv9vks5tN4VfgaJpZM4RqazS.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/pull/1748#pullrequestreview-91299682"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany","title":"geany/geany","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@b4n requested changes on #1748"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1748#pullrequestreview-91299682"}}}</script>