[RFC][PATCH] Change to GuiView::goToFileRow

[Stephan Witt]

Am 16.02.2020 um 14:48 schrieb Stephan Witt <st.witt at gmx.net>:
> 
> Am 16.02.2020 um 12:18 schrieb Enrico Forestieri <forenr at lyx.org>:
>> 
>> On Sun, Feb 16, 2020 at 01:47:43AM +0100, Stephan Witt wrote:
>>> Am 16.02.2020 um 01:25 schrieb Enrico Forestieri <forenr at lyx.org>:
>>>> 
>>>> On Sat, Feb 15, 2020 at 04:09:14PM +0100, Stephan Witt wrote:
>>>>> Am 15.02.2020 um 15:31 schrieb Enrico Forestieri <forenr at lyx.org>:
>>>>>> 
>>>>>> On Sat, Feb 15, 2020 at 02:41:19PM +0100, Stephan Witt wrote:
>>>>>>> 
>>>>>>> I came up with another path. IMHO easier to understand. 
>>>>>>> And it works with all variants of Qt and input.
>>>>>> 
>>>>>> Unfortunately, it confuses me more than the previous one.
>>>>>> Please, can you apply the attached debug patch and report the
>>>>>> results you obtain with both Qt 5.9 and 5.12?
>>>>> 
>>>>> Here you are:
>>>>> 
>>>>> version: 5.9.8
>>>>> file_name: /var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.aScQxky52159/lyx_tmpbuf0/Tutorial.tex
>>>>> abstmp:    /var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.rHrJLJU14712
>>>>> realtmp:   /private/var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.rHrJLJU14712
>>>>> 
>>>>> Qt version: 5.12.3
>>>>> file_name: /var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.aScQxky52159/lyx_tmpbuf0/Tutorial.tex
>>>>> abstmp:    /private/var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.BOxdOihSBXEp
>>>>> realtmp:   /private/var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.BOxdOihSBXEp
>>>> 
>>>> Something really fishy is going on here. Note that you get the *same*
>>>> file_name with both 5.9.8 and 5.12.3. This is already really strange,
>>>> but the stranger thing is that the last component of abstmp and realtmp
>>>> changes, becoming lyx_tmpdir.rHrJLJU14712 in one case and
>>>> lyx_tmpdir.BOxdOihSBXEp in the other one. Seemingly, they are unrelated
>>>> to file_name…
>>> 
>>> To simplify my testing I’m using the mini-buffer command input to debug the
>>> whole thing. I’ve used the arrow up in mini-buffer to repeat the previous command.
>>> Sorry for not telling you this.
>> 
>> Please, try instead triggering the debug output by actually compiling a
>> document and initiating a real reverse search.
> 
> 1. Complete cycle with Qt-5.9 + Debug + Patch
> 
> Format.cpp (727): Executing command: open -a Skim "Tutorial.pdf"
> Qt version: 5.9.8
> file_name: /var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.zXGLIYJ71248/lyx_tmpbuf0/Tutorial.tex
> abstmp:    /var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.zXGLIYJ71248
> realtmp:   /private/var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.zXGLIYJ71248
> frontends/qt/GuiView.cpp (3691): goToFileRow: row 670 in file /private/var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.zXGLIYJ71248/lyx_tmpbuf0/Tutorial.tex
> frontends/qt/GuiView.cpp (3707): goToFileRow: buffer lookup for /var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.zXGLIYJ71248/lyx_tmpbuf0/Tutorial.tex success
> 
> 2. With Qt-5.12 + Debug + Patch
> 
> Qt version: 5.12.6
> file_name: /var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.ypXTymmURmqL/lyx_tmpbuf0/Tutorial.tex
> abstmp:    /private/var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.ypXTymmURmqL
> realtmp:   /private/var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.ypXTymmURmqL
> frontends/qt/GuiView.cpp (3691): goToFileRow: row 1002 in file /private/var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.ypXTymmURmqL/lyx_tmpbuf0/Tutorial.tex
> frontends/qt/GuiView.cpp (3707): goToFileRow: buffer lookup for /private/var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.ypXTymmURmqL/lyx_tmpbuf0/Tutorial.tex success
> 
> 3. With Qt-5.12 + Debug
> 
> 14:47:51.928: Die Datei existiert nicht: /var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.RAbdVPKaoDsn/lyx_tmpbuf0/Tutorial.tex
> Qt version: 5.12.6
> file_name: /var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.RAbdVPKaoDsn/lyx_tmpbuf0/Tutorial.tex
> abstmp:    /private/var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.RAbdVPKaoDsn
> realtmp:   /private/var/folders/1x/zm63s22x7s591xrlgm4rl9v40000gn/T/lyx_tmpdir.RAbdVPKaoDsn
> 
> Stephan

The latest version of the mentioned patch I’ve used for 1. and 2. is

-------------- next part --------------
A non-text attachment was scrubbed...
Name: goToFileRow-3.patch
Type: application/octet-stream
Size: 2200 bytes
Desc: not available
URL: <http://lists.lyx.org/pipermail/lyx-devel/attachments/20200216/f55002f5/attachment.obj>