[Github-comments] [geany/geany] Push current position to navqueue before navigating back (#1537)

Vasiliy Faronov notifications at xxxxx
Fri Jul 14 11:11:10 UTC 2017


vfaronov commented on this pull request.



> @@ -192,6 +192,14 @@ static gboolean goto_file_pos(const gchar *file, gint pos)
 void navqueue_go_back(void)
 {
 	filepos *fprev;
+	GeanyDocument *doc = document_get_current();
+
+	/* If the navqueue is currently at some position A, but the actual cursor is at some other
+	 * place B, we should add B to the navqueue, so that (1) we go back to A, not to the next
+	 * item in the queue; and (2) we can later restore B by going forward.
+	 * (If A = B, add_new_position will ignore it.) */
+	if (doc && doc->file_name)

@codebrainz Can you spell this out a bit? Do you mean keeping the `if (doc)` condition but also adding a `doc != NULL` assertion?

-- 
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/1537#discussion_r127434875
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20170714/0d687aeb/attachment.html>


More information about the Github-comments mailing list