Re: M-x move-to-prev-xref seems to cross node boundaries, which may not be intended

[Gavin Smith]

On Thu, Apr 02, 2026 at 10:11:42PM +0800, aidan wrote:
> 4.
> 
> scroll-behavior=Page Only
> cursor-movement-scrolls=On
> 
> In this case, only move-to-next-xref follow the "Page Only" scroll behavior
> (i.e. just stops at node boundaries)
> 
> while move-to-prev-xref behaves like case 6 (see below).
> 
> This is the case described in the original report.
> 
> 
> 
> 
> Among the six cases, only the fourth is the one I consider problematic and
> should be fixed
> 

Thanks for the thorough testing.  It was indeed a bug:

diff --git a/info/session-cmd.c b/info/session-cmd.c
index 20821d194c..10df74767b 100644
--- a/info/session-cmd.c
+++ b/info/session-cmd.c
@@ -1825,7 +1825,7 @@ DECLARE_INFO_COMMAND (info_move_to_prev_xref,
               return;
             }
 
-          if (backward_move_node_structure (window, info_scroll_behaviour != 0)
+          if (backward_move_node_structure (window, info_scroll_behaviour) != 0
               || !strcmp (window->node->nodename, initial_nodename))
             {
               break; /* No earlier nodes in file, or we are back where we


I've committed this fix in the git repository.