Typos in synctex/synctex_parser.c

Bug #1982665 reported by Louis-Philippe Véronneau
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
qpdfview
Fix Released
Low
Unassigned

Bug Description

Hello,

I'm working through the Debian patches and it seems this one has not been forwarded yet. "uncomplete" is the obsolete form of the English word "incomplete" [1]. Nothing very important but ... ¯\_(ツ)_/¯

[1]: https://en.wiktionary.org/wiki/uncomplete

Revision history for this message
Louis-Philippe Véronneau (pollo-debian) wrote :
Revision history for this message
Adam Reichold (adamreichold) wrote :

Hello Louis-Philippe,

this code is vendored from [1] and so the patch should be applied there and then updated code should be pulled here. From a quick glance, it seems that this was already fixed upstream so I will pull the updated code from there.

Note that the patch looks the wrong way around, i.e. it removes incomplete to add uncomplete?

Also note that the vendored should not actually be used as SyncTeX is provided as a shared library nowadays which qpdfview should prefer if it is available via pkg-config at build time. The Debian should definitely do that instead of relying on the vendored code. (The vendored code is just there for other platforms.)

Regards,
Adam

[1] https://www.tug.org/svn/texlive/trunk/Build/source/texk/web2c/synctexdir/

Changed in qpdfview:
importance: Undecided → Low
Revision history for this message
Adam Reichold (adamreichold) wrote :

Updated the vendored sources to version 1.21.

Changed in qpdfview:
status: New → Fix Committed
Changed in qpdfview:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.