Various fixes suggested by cppcheck

Rename local variables the hide other ones: get_binary_path, find_python_binary

Use unsigned int for conversion from hex using sscanf().

FileName::checksum(), parsecmd (SystemCall): use explcit values rather
than variable which value is known.

Forgot to test aspell and hunspell support

Use const &

Follow some of the performance advice from cppcheck

Most of that is changing string to string const &.

Fix bug #11619.

Fix nullptr warnings

Minor renaming for clarity.

Fix bug #11636.

Directories are created when files are copied, so we do not need
to do it prospectively here.

Revert last two commits, which we pushed accidentally.

Whoops, forgot this part

Constify, largely for debugging purposes

Constify

Enable dissolve in undefined Flex inset context menu

This is a hackish special case, so I had to add a hack...

Fixes bug #8650.

Reset layout when inserting an inset over full paragraph(s)

1/ When inserting an inset over a selection, it makes sense if the
selection covers a complete or several paragraphs to reset the layout of
the paragraph that contains the inset to plain layout. In general the
inner inset will have the needed layout information and it does not
make sense to keep this information outside.

However, when the inset forces plain layout, it does not make sense to
do that and the outer layout is preserved.

Some code that duplicates what is done in pasteSelectionHelper is also
removed.

2/ Similarly, when dissolving an inset, do not reset the layout of the
first paragraph if the inset was alone in its own paragraph.

Note that this does not work as intended when change tracking is enabled.

Fixes bugs #10266 and #10369.

Allow to hide formats from menus

Fixes: #11657

ru/Tutorial.lyx: update by Yuriy

UserGuide: some typos fixed by Yuriy

Give error if undefined citations with Biblatex

This commit catches situations when one key in a multi-key citation
is not defined. This commit causes an error to be given, but the
name of the key that is undefined is not provided in the error
dialog.

This commit is consistent with bf99ece7.

For more information, and possible follow-up discussion (e.g., on
putting the key in the error dialog), see the following ML thread:

https://www.mail-archive.com/search?l=mid&q=20190908165644.qnz6xu5bm5eqiko6%40boogie

#6961 correct the big-endian conversion of UInt32

This is a patch from Patrick de Visschere. Thanks.

Minor update for supported-languages test.

Make preview skript Python-3 compatible.

Reset directory after os.chdir

Related to bug #10454

Port gnuplot2pdf.py to Python 3

Instead of wait(), use communicate(), as mentioned here:

  https://docs.python.org/3/library/subprocess.html

Otherwise, the process seems to hang as cautioned in the above URL.

Also, use byte strings.

ctests: update supported-languages tests.

Simplify user preamble.

Use common test document for Xe- and LuaTeX with polyglossia
and special one for languages only supported by XeTeX.

Update tagging patterns and comments.

Obs. APA: fix missing citations

The document now compiles and the ctests pass.

Fix origin path of three obsolete docs

ctests: invert a test failing due to babel bug

This test fails with an updated TL, although it passes on some older
systems.

Update test document for supported languages.

lyx2lyx: Fix revert_new_languages

ru.po: Update from Yuriy

Enable change tracking if paragraph break change

When a selection spans more than one line, we now check for whether
there is a change at one position after the last position of each
fully selected line.

This fixes #11629.

Partially revert 6b014ec7

Allow use for font MonomakhUnicode. The font is available in texlive
and making a symbolic link in ~/.fonts/fonts to point to the appropriate
directory makes the font available to the system too.

ctests: update unreliableTests

ctests: uninvert AA tests thanks to recent fixes

Thanks to 54fd0a70, all AA tests can be uninverted except for two
texF tests.

Hooked in one year too early

Fix encoding in aa class, which is preset via class options

Hook into nearby file format change.

J of Acoustical Society: add a space before cite

Without the space, the citation followed directly the word "in".

Autotests: Try to not use hard available fonts

Fix test BATCH_outline-beamer

Small ctest update.

J of Acoustical Society: fix missing citations

This restores compilation of the file and all ctests work as
expected.

Set buffer correctly when deleting column in math grid

This is a followup to 837bcbb0.

Fixes bug #11623.

Initialize num_bib_key_

Spotted by Coverity.

Fix totalheight reversion routine if there was no totalheight set

Tutorial: some style fixes by Yuryi.

Update of Russian po and Tutorial by Yuriy

Use acadian rather than canadien

This is what babel-french urges, and canadien currently is broken.

acadien seems to be there for at least 15 years.

Enable Canadian English and French in polyglossia

These are equivalent to usenglish and french in babel anyway.

ctest update: new and resolved failures after TL update, resoved lyxbug.

Correct index

lyx2lyx: correct placement of (new) local layout

This ended up inside modules sometimes.

#11642 add version of used python to configure.log protocol

lyx2lyx: fixup gloss reversion

Fix export with Japanese (platex) and non-TeX fonts.

Revert a change in c83765163f that inserts a spurious "}".

unicodesymbols: force "latin capital letter Schwa" also for utf8x.

ctests: make a comment more specific.

ctest update: we now work around a problem with Czech or Slovak text in some tables.

Properly terminate command

de.po

Cmake tests: Really ignore wrong_output_3059-rtl-tables-xetex_pdf4_texF test

Handle endnotes aux file in removeAuxiliaryFiles()

As observed on users list it's hard to guess where to setup external editor.

Correctly set language after intitle paragraphs

Fixes: #11514

ja/EmbeddedObjects: fix case of KOMA-script cites

Without these case changes, the citations are undefined.

ja/EmbeddedObjects: update format for next commit

The format was already post-2.3.x.

Tiny cleanup

No change in functionality

ru/Tutorial: re-save in 2.3 format

ru/Tutorial: fix for utf8 doc encoding

es/UserGuide: comment out 2 citations to fix comp

The citation "spanish" is broken. Commenting it out at least fixes
the compilation.

es/UserGuide: fix a broken cross-reference

es/Customization: fix ref to fix compilation

The reference was using the label of the English version, so was
broken.

es/Customization: update format for next commit

This manual was already saved with a post-2.3.x format, so I am
updating to the format of master so that the next commit's diff is
readable.

es/Additional: remove citation to fix compilation

Removing this last citation in the manual is consistent with
032932cb, as well as other similar places in the text (it seems we
now reference documentation with a hyperlink instead of a citation).

CREDITS: "!" in email should be followed by space

Otherwise a literal "!" is output. This commit fixes typos in two
email addresses.

hebarticle: fix compilation with XeTeX (bug #10525)

Fix coloured boxes in RTL with [pdf]latex (#8642)

Fix greyedout note in RTL (#8647)

More complicated than I wished for, but color handling in RTL is
particularly tricky.

The three versions are needed due to the differences between [pdf]latex,
xetex, luatex/luabidi and luatex/babel.

Implement commenting for CREDITS file

The reader of CREDITS (GuiAbout.cpp) now ignores lines that start
with the number sign (#). This commit also introduces a commented
out header in CREDITS reminding any potential editor that all
changes will be overwritten by generate_contributions.py.

Port CREDITS changes to source

All changes must be in generate_contributions.py.

This commit amends 144e5189.

CREDITS: remove errant ">" from two addresses

Correct typo in contributor email address

ar.po: update from Hatim Ali

Correct ^ catcode for \cprotect

Update tex2lyx tests

InsetGraphics: use totalheight for height output

Graphics "height" is only the height above the baseline and thus not
what most people will expect. Also, using height can result in a
division by zero with rotation.

Use totalheight now since this means "height of the figure" (independent
of the baseline).

The custom options field can be used if height is really desired.
We handle this in conversion/reversion, respectively, so document
output will not change.

Hence, file format change.

Fixes: #9676

Load rotfloat and rotating after graphicx.

Fixes: #11502

Re-fix #2005

Loading keyval does not work any longer. Fix it properly by de-activating
the equal character locally if graphics options use it.

Fix cprotect with Slovak

babel-slovak makes ^ active, though cprotect relies on it being
catcode 9 ("ignored"). Thus we locally change the catcode with Slovak.

Deactivate active - in tables with \cline or \cmidrule

This introduces a new languages tag ActiveChar which also can be used
for similar cases.

Fix cline/cmidrule calculation once more

fix midrule calculation

GuiSetBorder: Fix drawing glitch with trimmed lines

Don't output default papersize

Fix another dead ref

Fix another dead ref

Fix documents

Update sk.po

Update sk.po

Support memoir's builtin endnotes

Localize endnotes header (with endnotes.sty)