ctest autotest for #11773 language nesting with lyx2lyx.

* invert failing lyx2lyx tests for ko/Welcome
* add dedicated test sample
* set language for English text part in ko/Welcome.

Also

* fix a lyx2lyx language test sample
* fix clause in unreliableTests

Remove no longer needed module import.

Refactor lyx2lyx code.

Use helper functions for shorter, self-documenting code.

Drop optional arguments that equal the default value.

Remove hard TABs and trailing whitespace.

Fixup cf9827df: do not forget keyboard bindings and docs

Keyboard bindings are easy. For lfun icons in documentation, both the UserGuide and Tutorial have to be updated. Since this will have to be done again once the documentation is imported from branch, I list them here for reference:

For the lib/doc/ directory:
find . -name 'Tutorial.lyx' | xargs sed -i 's/"layout List"/"layout Labeling"/g'
find . -name 'UserGuide.lyx' | xargs sed -i 's/"layout List"/"layout Labeling"/g'
and then
find . -name 'Tutorial.lyx' | xargs sed -i 's/"layout \(Itemize\|Description\|Labeling\|Enumerate\|List\)"/"layout-toggle \1"/g'
find . -name 'UserGuide.lyx' | xargs sed -i 's/"layout \(Itemize\|Description\|Labeling\|Enumerate\|List\)"/"layout-toggle \1"/g'

Related to bug #11770.

List is obsoleted by Labeling: update icons

This does not change much in master but it is cleaner since the List
layout has been obsolete since 2010 (7f8b868f).

See bug #11770.

Fix bug #10316.

By sending the request through the usual dispatch machinery, we
make sure the cursor is valid when we're done.

Andr'e points out this is wrong. Of course it is, since that is not a pointer.

Do not display caret when we're not ready

Using a dialog may trigger a redraw at times where the metrics caches
have not been updated yet. To fix this, do as in paintEvent() and
abort caret blinking if there is an open undo group.

Move the decision to skip the caret painting to showCaret(), closer to
where real work happens. hideCaret () is now an alias for
showCaret(false), which allows some code refactoring.

See also commit c7496a11b2.

Fixes bug #11763 (although it does not trigger on master).

Fix nullptr warning

Send a DispatchResult object with dispatch requests to the TOC.

This allows us to request a buffer update.

Amend(1) 7f0efc51: Add evince_sync files to the dist target of automake

Fix nullptr warnings

Cmake build: Add evince_sync-files to be installed under the scripts dir

Moving 3rdparty/evince_sync to 3rdparty/scripts/evince_sync

Cmake build: Remove deprecated policy warning

Cmake tests: Amend f6f51587

Un-provide amsmath in beamer article layouts

Fixes #11768

Avoid a copy.

ru.po: update by Yuriy

Fix warnings and use ranges.

Fix warnings

Fix a bunch of warnings

Fix warnings in support/.

Fix header.

Null pointers.

Cmake build: Use correct include if using internal hunspell

This way also "HAVE_HUNSPELL_CXXABI" is correctly set to 1

Use ranges, fix warning

Use ranges, fix warning

Use range

Fix nullptr warnings

Use ranges, fix warnings

Fix nullptr warnings

Use ranges, fix warnings

Fix warnings

Fix warnings

Use range

Fix warnings

Use range and rename private variable

Use more ranges

Use ranges

Polish

More requires --> required, for C++2a.

Apparently, "requires" is a keyword in C++2a.

Remove code that has not been used since 2010.

See 208acbc72.

Stop dialogs from updating twice all the time.

I've long wondered why dialogs seem to update twice when it seems
they just need to update once. It seems it was this code. But it
was only put there to deal with a problem with code in the button
controllers that was disabled in 2007! So I'm going to try removing
it. If we start seeing weird behavior with dialogs in master, then
this will be why.

Fix warnings

Remove code that has not been used for 13 years.

Fix warnings.

Fix a bunch of warnings.

Revert unintended change

Fix null pointer warnings.

Fix warning (and simplify)

Fix bug #11748.

Track the last chosen citation style when we clear the entries.

Fix a bunch of warnings

Fix problem with validation when using InsetLayout.

Cleaner fix to ensure 32-bit XCB events

This fix still satisfies Valgrind and is cleaner than the approach
at 19c41bd0: instead of using calloc we now use the C++11 specifier
alignas. For more info, see the following ML thread:
https://www.mail-archive.com/search?l=mid&q=20200219024908.5n4x4osni55gylo3%40tallinn

Revert "Fix write to uninitialized bytes for XCB event"

This reverts commit 19c41bd09572f15ac3546c8e6a6d4ec15306c93a.

A more efficient fix is on the way.

Fix bug #11750.

Unfortunately, stat.st_ino returns 0 on Windows, at least on Python 2.7, so we can't use that way of telling when we're seeing the same directory again. Surely the real pathname should work.

Fixup d207e85c: paint sublabel with the correct background color

The above mentionned patch did paint the background of the sublabel to
avoid "bold-like" effect. However the correct backgound color is the
row's one, not the inset's one.

Fix alignment warnings from clang.

Use C++11 construct.

Fix FileName::realPath() on Windows

The realPath() implementation on Windows works only for files and
not for directories. By using an API available starting from Vista
it is possible to fix it in a simple way.

I also took into account using the Qt QFileInfo::canonicalFilePath(),
but it turns out to not work when a path component is a junction
(tested with Qt 5.14.1).

Due to this, it is not possible compiling or using LyX on Windows
versions earlier than Vista.

Try again to fix memory leak reported by Scott.

This time, use a shared_ptr.

Revert "Fix memory leak reported by Scott."

It leads to a double delete error.

This reverts commit 11397a2e82ba19b69818f9891132ff7074bc1759.

Fix memory leak reported by Scott.

Fix some clang warnings.

The first one, a comma instead of a semi-colon, is a weird one.

Fix nullptr warnings

Fix "no extern declaration" warning.

Cmake build: Remove unused option

LYX_USE_FILEDIALOG is not used since af795b80

Make sure that spelling marks do not spill ouside of row

This avoids annoying remnants of dotted red lines.

Fixes bug #11755.

Avoid UTF8 "no-break space" and use the space inset instead.

Provide predefined key binding for LFUN "forward-search" on Mac

Use real path names for buffer lookup by name of temporary external files.

In case of path names for external files containing symbolic links the real path
and the logical path name may be different for the same file or directory.
LyX is using QDir::tempPath() to create the path name of the temporary directory.
The Qt implementation is free to return the logical or the real path name here and
it happens to be different for various platforms and versions.
The most stable and clean solution is to use the real path name consistently.

Make the directories in LyX application bundle read only.

Python produces files with cached on-the-fly compiled code on disk inside the bundle.
This should be avoided in case of a signed application bundle.

Cleanup code signing of the LyX application bundle

Cleanup internal structure of embedded Qt frameworks

Improved standard conformance of private frameworks naming scheme

Avoid potential leak of memory pointed to by QTreeWidgetItem subcatItem

Add packaging option to code sign the LyX application

Create private frameworks on Mac with standard conforming internal structure

gcc 4.7 leftover

Only gcc 4.8 is C++11 feature complete.

Avoid potential leak of memory pointed to by QTreeWidgetItem catItem

ru.po: update by Yuriy

Fix write to uninitialized bytes for XCB event

As the xcb_send_event man page [1] states,

  In order to properly initialize these bytes, we allocate 32 bytes
  even though we only need less for an xcb_configure_notify_event_t

This commit fixes the following Valgrind error, which could be
triggered by selecting a letter in LyX:

  ==12698== Syscall param writev(vector[...]) points to uninitialised byte(s)
  ==12698==    at 0x61F578D: __writev (writev.c:26)
  ==12698==    by 0x61F578D: writev (writev.c:24)
  ==12698==    by 0x4A83BFC: ??? (in /usr/lib/x86_64-linux-gnu/libxcb.so.1.1.0)
  ==12698==    by 0x4A83FD0: ??? (in /usr/lib/x86_64-linux-gnu/libxcb.so.1.1.0)
  ==12698==    by 0x4A84246: ??? (in /usr/lib/x86_64-linux-gnu/libxcb.so.1.1.0)
  ==12698==    by 0x4A84ACB: xcb_flush (in /usr/lib/x86_64-linux-gnu/libxcb.so.1.1.0)
  ==12698==    by 0x17C8F06: lyx::frontend::GuiApplication::nativeEventFilter(QByteArray const&, void*, long*) (GuiApplication.cpp:3366)
  ==12698==    by 0x5AA4EEE: QAbstractEventDispatcher::filterNativeEvent(QByteArray const&, void*, long*) (qabstracteventdispatcher.cpp:484)

[1] https://www.x.org/releases/current/doc/man/man3/xcb_send_event.3.xhtml

Fix memory leak.

Spotted by Scott.
https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg210447.html

Add missing check for nullptr on check of language encoding of next paragraph.

Add missing initialization for local variable.

The autoargs variable is random for empty macro argument lists otherwise.

Remove useless assignments to a local variables never read later.

Remove useless assignment to a local variables never read later.

Avoid crashes in case of missing document buffer for buffer based conversions.

Silence some clang warnings.

Fix a strange warning in QtCreator:

/cvs/lyx/lyx-devel/src/frontends/qt/TocModel.cpp:356: warning: loop variable 'toc' has type 'const pair<std::string, shared_ptr<lyx::Toc> > &' (aka 'const pair<basic_string<char>, shared_ptr<vector<lyx::TocItem> > > &') but is initialized with type 'const std::pair<const std::__cxx11::basic_string<char>, std::shared_ptr<std::vector<lyx::TocItem, std::allocator<lyx::TocItem> > > >' resulting in a copy

/cvs/lyx/lyx-devel/src/frontends/qt/TocModel.cpp:356: use non-reference type 'pair<std::string, shared_ptr<lyx::Toc> >' (aka 'pair<basic_string<char>, shared_ptr<vector<lyx::TocItem> > >') to keep the copy or type 'const std::pair<const std::__cxx11::basic_string<char>, std::shared_ptr<std::vector<lyx::TocItem, std::allocator<lyx::TocItem> > > > &' to prevent copying

Easier just to use 'auto'.

Remove unnecessary assertion after explicit buffer pointer check.

Avoid potential crashes with missing document buffer in dispatch of buffer based LFUNs.

Use LASSERT with appropriate action instead of LATTEST assertions.
Debug build is unchanged and asserts, release build skips the action gracefully.

Code cleanup: avoid repeating temp_dir method calls while local variable has a result already.

Restore custom tmp directory

Fixes #1174

lyxpreview_tools.py: fix a regular expression

Three backslashes are needed before a LaTeX command, not one. Before
this commit, the code gave the following error with Python >= 3.6:

  re.error: bad escape \g at position 29

This error was introduced with Python 3.6, as documented [1] by the
following line of documentation:

  Changed in version 3.6: Unknown escapes in pattern consisting of
  '\' and an ASCII letter now are errors.

Although previous Python versions did not give an error, the regular
expression was not working as intended: for example, the "\\n" in
"\\newcommandx" would be interpreted as a new line.

[1] https://docs.python.org/3.6/library/re.html#re.sub

Unify handling in dispatchVC.

All these check could be actually dropped, because unless dispatchVC
is called directly, getstatus would already handle it.

Amend 43dfd12f