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.

Fix cut and past error (reported by coverity)

fix potential null pointer (reported by coverity)

* postats.py

Remove handling of dispatched() property in BufferView::dispatch

As indicated in DispatchResult.h, this is only needed in insets'
doDispatch() methods, where lfuns can be delegated to a higher lever.

Add ScrollType BOTTOM and TOGGLE for BufferView::scrollToCursor()

BOTTOM shows the paragraph containing the cursor at the bottom of the
work area.

TOGGLE does CENTER, then TOP, BOTTOM and finally cycles to CENTER.
This is a feature copied from emacs's C-l binding.

Add new argument "caret" to lfun "scroll" that can be used like
  scroll caret center

Change the keys (either C-l, M-l or F5) bound to screen-recenter to
"scroll caret toggle".

As an experiment, change paragraph-goto to use this scrolling method.
This can be tested when going to a heading in the outline pane.

Cleanup Page Up/Down code

Since BufferView::scroll() does not correct scrolling amount at
top/bottom anymore, remove its return value and add a new one to
BufferView::updateMetrics(bool) that returns this correction.

Rely on updateMetrics(false) to set the metrics straight after moving
the anchor y position.

Use assert_in_view = false when calling TextMetrics::editXY() because
this parameter does not work as advertised: if an inset not totally
visible, the code will not try to go inside it to look for a smaller
row that is totally visible.

Two small (hopfully useful) modifications:
- take into account the cursor X target instead of its screen position
- adapt the cursor position depending on
  lyxrc.mac_like_cursor_movement when hitting start or end of
  document.

Get rid of BufferView::scrollUp/Down

LFUN_SCROLL was the last user, change it to use only scroll(). Take
this opportunity to improve a bit this dispatch code.

Also improve somewhat the methods documentation.

Streamline BufferView::scrollToCursor

There are two parts in this patch :

1/ in the part where the cursor is already visible, replace calls to
scroll/scrollUp/scrollDown by direct manipulation of the anchor
vertical position. Reasoning in terms of new position rather than
amount of pixels to scroll makes work easier.

2/ in the part where the cursor is set to a new TOP or CENTER
position, move the code around in order to prepare for new ScrollType
enums.

No change intended, except that CENTER now relies on the contents
dimension of the row that holds the iterator, that is without spacing
and labels.

Simplify greatly Buffer::scroll/Up/Down methods

The code there that checks whether we scroll beyond the top/bottom of
document is not needed anymore, this is handled later by
BufferView::update(bool).

Note that this methods may disappear with time.

No change intended.

Fix update of cursor in tab-delete when there is a selection

The position of the cursor should not be decreased if it is already at
the start of the paragraph. This can lead to a crash when trying to
display the caret.

Fixup 999fb37e: fix thinko

The current buffer is part of allRelatives(), contrary to what I
thought. Therefore updateId() increased the buffer id by two.

update comments

Remove legacy code to compute geometry

This code is not necessary anymore after the fix to #13069.

Remove the first QString argument to createView(). Set the autoShow
parameter to false by default and rename createView(id) to
createAndShowView(id) to avoid member function overloading issues.

Remove optional <GEOMETRY> argument of window-new lfun.
Document the removal of the argument in release notes.

Amend 18c310a8 for cmake build: Get rid of QPA_XCB define

Get rid of QPA_XCB define

Since the platform is selected at run time (e.g. xcb vs. Wayland), it
is not a good idea to decide at compile time what code is run. Another
example is on macOS, where having xcb libraries available will lead to
lauching the Xquartz whenever LyX is running.

With this patch, things are separated:
- the code is run when run-time platform is "xcb".
- the support code is compiled in when xcb header and libraries are
  available.

Fixes ticket #13086.

Streamline -geometry command-line option support

Instead of baking our own support for -geometry command line option on
Windows (Qt only does that for Xcb), use the fact that Qt 5 provides
the equivalent -qwindowgeometry flag.

All we need to do is to replace the -geometry option with -qwindowgeometry.

This is part of the effort in ticket #13086 to get rid of the
compile-time QPA_XCB define.

Amend f1fddd86

Add defines for HAVE_XCB_XCB_H and HAVE_LIBXCB to cmake build

Cmake build: Add defines for HAVE_XCB_XCB_H and HAVE_LIBXCB

Add missing Spanish modifier

Force a metrics update of buffer after reload

It makes reload more robust and is in any case good enough to avoid
crash in #139091.

Fixes bug #13091.

Update it.po

Update sk.po

Reset lang_options before reading headers

output babel options via \babelprovide in all cases needed

Amend fc605cf138c

Use proper option if main and a secondary language have BabelProvide 2

Another language GUI string improvement

de.po

Clarify usablity of language support variants in GUI string

Farsi and Arabic need BabelProvide 2, too

Properly set the babel bidi option (#12862)

Add proper support for BabelProvide 2

We need to instruct babel not to load an existing ldf in this case

Fix typo

de.po

Fix GUI string

Fix typo in UI file

Update tex2lyx tests

Avoid empty option in babelCall

Add support for the following languages

Polyglossia:
Chinese (Simplified), Chinese (Traditional), Japanese, Kurdish (Sorani),
Latin (Classic), Latin (Ecclesiastic), Latin (Medieval), N'ko, Odia,
Punjabi, and Uyghur

Babel:
Amharic, Armenian, Asturian, Bengali, Church Slavonic, Coptic, Divehi,
Kannada, Kazakh, Khmer, Kurdish (Sorani), Lao, Latin (Classic),
Latin (Ecclesiastic), Latin (Medieval), Malayalam, Marathi, N'ko,
Occitan, Odia, Punjabi, Russian (Petrine orthography), Sanskrit, Syriac,
Tamil, Telugu, Tibetan, Urdu, and Uyghur

Python 3.13 removed the pipes module from the standard library

That module was a wrapper over the popen module and it was scheduled to
be removed in Python 3.11.

This code is also compatible with Python 2.

Update sk.po

Assure OK/Apply button is initially disabled in InsetParamsDialogs (#8927)

Some more language TODOs

Obviously, we can support all languages that have so far only been
provided via polyglossia with babel(provide)

Load Ancient Greek and Mexican Spanish via \babelprovide

In classic babel, these varieties are only available via modifiers
or attributes, which means that they could only be used instead rather
than in addition to the default variety

Basic support for \babelprovide

This is the new(er) babel mechanism to load languages via babel *.ini
files.

Some languages (or varieties) are only supported this way, some work
better with unicode engines this way (for the latter case, we have
BabelProvide 2)