Work around bug in aa.cls

Factor out (signed) glueLenghValidator

Allow negative offset in GuiLine (#13102)

UG: document hyperref borders/colors interaction.

https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg222962.html

Re-focus workarea after some outline context-menu actions (see #1624)

Thanks Pavel for the main idea

QFontMetrics::xHeight() is not always the ascent of 'x'

Use the real ascent of glyph 'x' as LyX 2.3.x did.
It can make a difference at least on Windows with many fonts.

Fixes bug #13100.

Do not use big characters for some unicode points

This modifies LOGICAL AND, LOGICAL OR, INTERSECTION and UNION.

Fixes bug #13099.

Better fix for Spanish Tufte problem

The option "nohyper" causes the issue

Revert "Work around fragility problem in Spanish Tufte Handout"

This reverts commit 829845457753c1348f9b0399238b999c527a7d60.

Work around fragility problem in Spanish Tufte Handout

It is unclear what the actual problem is; it only shows up in this
document (not the other l7ns)

Cmake tests: Amend 6e05409d

Use with a slightly better thought out selection of possible exports to tex-source.

Update sk.po

Cmake build: Omit "PRE_BUILD" in add_custom_command()

According to cmake warning output while configuring:
CMake Error (dev) at po/CMakeLists.txt:72 (ADD_CUSTOM_COMMAND):
  The following keywords are not supported when using
  add_custom_command(OUTPUT): PRE_BUILD.

  Policy CMP0175 is not set: add_custom_command() rejects invalid arguments.
  Run "cmake --help-policy CMP0175" for policy details.  Use the cmake_policy
  command to set the policy and suppress this warning.
Call Stack (most recent call first):
  po/CMakeLists.txt:106 (add_gettext_python)
This error is for project developers. Use -Wno-error=dev to suppress it.

ctests: invert chess lyx2lyx tests

In export.cmake, if the detected output format is pdf3, we try to
export to "platex", which usually makes sense, but for this file we
would need to export to "latex". This file is very particular, and
also old, so I do not know if it is worth the time to handle this
case.

This commit inverts the following tests:

  export/examples/Articles/Chess/Game_1_lyx16 (Failed)
  export/examples/Articles/Chess/Game_1_lyx20 (Failed)
  export/examples/Articles/Chess/Game_1_lyx21 (Failed)
  export/examples/Articles/Chess/Game_1_lyx22 (Failed)
  export/examples/Articles/Chess/Game_1_lyx23 (Failed)

de.po

Only \protect labels in \thanks notes

See https://marc.info/?l=lyx-users&m=136394387505489

GuiBibTeX: introduce way to toggle between absolute and relative local paths (#13097)

Header not needed anymore

Move assertion at the right position

It would wrongly trigger when doing a multi-cell selection.

ctests: invert lyx2lyx tests that fail TeX export

These fail when exporting to TeX, but since they have to do with old
formats, it's not clear it's worth the time to investigate.

These started failing once we check the exit code of LyX's TeX
export (at c7be9780).

ctests: exit with error if export to .tex fails

Now that we detect and export to the correct TeX format (instead of
assuming pdflatex), we can make the check more strict by giving an
error if LyX exits with error from the export.

ctests: default for Japanese + systemF is pdf4

This fixes the following tests:

  export/examples/ja/Modules/Rnw_%28knitr%29_lyx22
  export/examples/ja/Modules/Rnw_%28knitr%29_lyx23
  export/examples/ja/Modules/Sweave_lyx22
  export/examples/ja/Modules/Sweave_lyx23

Fix outliner tree depth (#13098)

Move header where needed.

ctests: comment the inverted ja Math systemF tests

Comments from Jürgen about why it is complicated to fix the exports.

Amend e72ca6a5

Header not needed anymore

Cmake lyx2lyx export tests: Try to use output format from the lyx-source

Do not output empty statement in lyx2lyx reversion

Uninvert ja UserGuide systemF tests

These exports pass thanks to Jürgen's recent fixes, including
working around a language nesting issue.

ja/UserGuide: work around language nesting issue

Uninvert the ja Additional systemF tests

These exports now succeed thanks to Jürgen's recent fixes.

Add missing ("false") condition

Update sk.po

amend 8e49c9a704b941fe5

we might not be using babel, after all

Fix es/ModernCV on older versions that do not have language options

Fix most math problems with Xe/LuaTeX

One NULL character issue left

Fixup ja/Additional for Xe/LuaTeX

Fixup ja/UserGuide for XeTeX

The system fonts do not cover Western quotation marks

Fix font def

A&A does not work with polyglossia

(babel is hardcoded in the class)

Uee a more accesible JP mono font

Fixup polyglossia-languages test file

* Workaround for churchslavonic

Fixup polyglossia-languages test file

* Branch Kazakh as it is now babel-only
* Setup fonts for Chinese via fontspec
* Setup proper fonts for Japanese
* Use proper Japanese language variant

Fix Spanish ModernCV example

Set appropriate system fonts for Chinese docs

More places where DocumentClassPtr should not be copied

Spotted by COverity scan.

Pass sigPtr object as const reference too

Spotted by Coverity scan.

Pass SocketCallback object as const reference

Spotted by Coverity scan.

Set appropriate tt font that supports CJK

The actual font should be discussed. This one is OSS and works well:
https://github.com/adobe-fonts/source-han-code-jp

Fixup some more Japanese templates

All of these rely on non-polyglossia language support

Fixup Japanese Article template

This one relies on its own language support

Fixup Japanese Feynman example

Polyglossia is loaded after the user preamble

Fixup Japanese Article example

This one relies on its own language support

Fixup test file for texF tests

Use proper fonts basically (the rest is format conversion)

Revert "use std::move() instead of copying when value is no longer used"

This reverts commit e685d31f5aadadd9c84c80a604818d49f5853dbf.

Revert "Fix compiler warning"

This reverts commit 0a7585403832c114fd2d084363aee98c3234058e.

Transform CoordCache::check/checkDim in ASSERT_DIM/POS macros

This makes it more obvious to the reader that these are actually
assertions, and should help Coverity scan too.

Pass shared_ptr<> arguments by const reference

These arguments are not expensive to copy. However, it makes sense to
pass such pointers by const reference when they are just intended for
reading.

Some reading about this issue is here:
https://www.internalpointers.com/post/move-smart-pointers-and-out-functions-modern-c

Fixes some Coverity scan defects.

Pass DocumentClassConstPtr arguments by const reference

This arguments are shared_ptr objects, so they are not expensive to
copy. However, it makes sense to pass such pointers by const reference
when they are just intended for reading.

Some reading about this issue is here:
https://www.internalpointers.com/post/move-smart-pointers-and-out-functions-modern-c

Fixes some Coverity scan defects.

Make two parameters const references

Fix uninitialized variable

Spotted by Coverity scan.

Do not use rand() to set a BranchList id

Use a simple counting instead, beecause Coverity complains that rand()
is not safe, and counting is siimpler anyway.

Make string parameter a const reference

Spotted by Coverity scan.

Avoid a couple of copies.

Spotted by Coverity scan.

Make docstring parameters const references

Spotted by Coverity scan.

Graph::getReachable(): pass a parameter by const reference

Spotted by coverity.

Avoid a copy

Spotted by Coverity scan.

Avoid a couple of copies

Spotted by Coverity scan.

GuiWorkArea: pass KeySymbol parameters as const references

Spotted by Coverity scan.

Avoid a couple copies in Trans::addDeadKeys().

Spotted by Coverity scan.

TransState::addkey() pass parameter by const reference

Spotted by Coverity scan.

Pass three string parameters by address

Spoted by Coverity scan.

Pass a docstring parameter by address

Spotted by Coverity scan.

Pass a pair of docstring arguments by address

Spotted by Coverity scan.

Avoid an unnecessary string copy

Spotted by Coverity scan.

Pass arguments by address in convert() templates

The situation of the convert<>() template was a mess: the header
announced that template specializations passed strings by address (my
doing long ago), but the code would just define versions that upass
strings by value.

The solution is to pass all values by address. This works very well,
_except_ when the value is a string litteral.

Defining the templates thus requires a bit of work.

Spotted by Coverity scan.

Fix linking of test programs in support/.

Pass parameters of constructor by address

Spotted by Coverity scan.

Fixup 716e20ae: revert useless part

Limit the number of iterations for some tabular features in math

Try to please Coverity scan.

Make some code that handle authors more readable

Create accssors authodmap() in BufferParams.

Use [] operator instead of find().

Hopefully this will avoid to confuse Coverity scan.

update test

Whitespace

fix jlreq abstract layout and adapt Chinese docs for unicode engines

jlreq layout defined the abstract environment to be boxed
in the preamble. Doing any kind of typesetting before
all the setup has finished is error prone and may lead to
wrong output (wrong fonts for example). The correct
place to put the abstract code is right before \maketitle.

The Chinese docs loaded the ctex package in case system fonts
were used. Now, with the new support for languages with babel
and polyglossia it is no longer necessary and creates collisions.

Do not push options to \babelprovide which do not make sense there

Should fix a couple of tests

Add a comment for Python binary names.

Improve coverity modeling

This tells coverity that these functions terminate the program.

Do not forget to reset stream after std::hex

Spotted by Coverity scan.

Do not handle special case where buffer == NULL

If it is so, we have already crashed in isBranchSelected() above.

(and hopefully, buffer_ is never null these days).

Spotted by Coverity scan.

Do not access current_view_ if it is NULL

Spotted by Coverity scan.

Initialize client's name in main()

Initializing clientName at declaration time is not a good idea.

Spotted by Coverity scan.

Fixup b8cdfd1e1db: fix thinko

Spotted by Coverity.

Do not use lyxerr in from_ascii(char const *)

This function is used a lot to initialize static docstrings. This is
an issue in theory because lyxerr may not have been initialized in
time.

In practice of course, lyxerr will only be used when there is a
non-ascii character in the parameter, which is really not supposed to
happen.

This gets rid of 75 coverity reports.

adapt tex2lyx to 0beb790a6a194

Handle space issue with refstyle more downstream

It is not necessarily, and infers with some usages, to generally escape
spaces in labels and references

Fix indentation

Fix compiler warning

Fixes the following warning:

  warning unqualified call to 'std::move' [-Werror,-Wunqualified-std-cast-call]

initialize boolean (reported by coverity)

use std::move() instead of copying when value is no longer used

This is mainly to shut down coverity which nowadays nags on this.
I am not sure whether we really gain something in these cases.