bug#19157: 25.0.50; Horizontal auto-scrolling

bug#19157: 25.0.50; Horizontal auto-scrolling

[Dani Moncayo]

[-- Attachment #1: Type: text/plain, Size: 532 bytes --]

Recipe:
  emacs -Q
  M-x toggle-truncate-lines RET
  ; Visit the attached file
  Type: C-s t l s . c C-s C-s


After the third 'C-s', the cursor is again over the original matching
word but we can't see it.  Emacs should do horizontal auto-scrolling
during Isearch, just as it does vertical auto-scrolling.

In GNU Emacs 25.0.50.5 (i686-w64-mingw32)
 of 2014-11-23 on LEG570
Windowing system distributor `Microsoft Corp.', version 6.3.9600
Configured using:
 `configure --without-dbus CPPFLAGS=-I/mingw32/include'

-- 
Dani Moncayo

[-- Attachment #2: test --]
[-- Type: application/octet-stream, Size: 637 bytes --]


gcc -std=gnu99  -c -mtune=pentium4  -DGLYPH_DEBUG=1 -DUSE_CRT_DLL=1 -I /C/msys/home/Dani/emacs/trunk/nt/inc -Demacs  -I. -I/C/msys/home/Dani/emacs/trunk/src  -I../lib -I/C/msys/home/Dani/emacs/trunk/lib   -mtune=pentium4     -mms-bitfields -Ic:/mingw/include/librsvg-2.0 -Ic:/mingw/include/glib-2.0 -Ic:/mingw/lib/glib-2.0/include -Ic:/mingw/include/gdk-pixbuf-2.0 -Ic:/mingw/include/cairo -Ic:/mingw/include/libpng16 -Ic:/mingw/include/pixman-1      -Ic:/mingw/include/libxml2               -MMD -MF deps/gnutls.d -MP  -Ic:/mingw/include -Ic:/mingw/include/p11-kit-1       -g3 -O2 -gdwarf-2  /C/msys/home/Dani/emacs/trunk/src/gnutls.c

bug#19157: 25.0.50; Horizontal auto-scrolling

[Dani Moncayo]

> Recipe:
>   emacs -Q
>   M-x toggle-truncate-lines RET
>   ; Visit the attached file
>   Type: C-s t l s . c C-s C-s

Opps, as you surely have noticed, the intended recipe is this one:

   emacs -Q
   ; Visit the attached file
   M-x toggle-truncate-lines RET
   Type: C-s t l s . c C-s C-s

Sorry.

-- 
Dani Moncayo

bug#19157: 25.0.50; Horizontal auto-scrolling

[Eli Zaretskii]

> Date: Mon, 24 Nov 2014 09:29:29 +0100
> From: Dani Moncayo <dmoncayo@gmail.com>
> 
> > Recipe:
> >   emacs -Q
> >   M-x toggle-truncate-lines RET
> >   ; Visit the attached file
> >   Type: C-s t l s . c C-s C-s
> 
> Opps, as you surely have noticed, the intended recipe is this one:
> 
>    emacs -Q
>    ; Visit the attached file
>    M-x toggle-truncate-lines RET
>    Type: C-s t l s . c C-s C-s
> 

Fixed in 9faf591 on master.

The incorrect code in isearch.el was there since about forever.  I
have no idea why this problem is only visible on the trunk; I suspect
some recent redisplay optimization exposed it.  If you have time to
bisect, perhaps we will learn something non-trivial.

Thanks.

bug#19157: 25.0.50; Horizontal auto-scrolling

[Eli Zaretskii]

> Date: Mon, 24 Nov 2014 15:36:15 +0200
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: 19157@debbugs.gnu.org
> 
> The incorrect code in isearch.el was there since about forever.  I
> have no idea why this problem is only visible on the trunk; I suspect
> some recent redisplay optimization exposed it.  If you have time to
> bisect, perhaps we will learn something non-trivial.

Using old binaries I kept around, I was able to determine that the bug
appeared somewhere between 26 July and 27 July 2014 (bzr trunk
version 117575 works correctly, while 117589 does not).  That range of
revisions includes the pixelwise-resizing changes...

bug#19157: 25.0.50; Horizontal auto-scrolling

[Dani Moncayo]

> Fixed in 9faf591 on master.

Confirmed.  Closing.

Thanks!

-- 
Dani Moncayo