CMake build: find enchant 2.x binary and lib

For example, see the Fedora file lists for the package
enchant2-devel [1] as well as the Debian Sid file lists for the
packages libenchant-2-dev [2] and enchant-2 [3]. The packages for
Ubuntu 20.04 are similar. Our autotools build has had support for
finding enchant 2.x since 71977769.

Searching for enchant-2 first is consistent with what we do for
autotools (see 1ae537c5).

This commit is related to #10986.

[1] https://www.lyx.org/trac/ticket/10986#comment:28
[2] https://packages.debian.org/sid/amd64/libenchant-2-dev/filelist
[3] https://packages.debian.org/sid/amd64/enchant-2/filelist

Improve performance of includeonly with "maintain counters and references"

First, we do not need to run bibtex/biber on the maintenance run, as
the necessary references will be generated on the includeonly run.

Second, exclude the master from DepTable in maintenance run, as the
master is re-checked in any case in the includeonly run, and as it will
always be detected as changed due to the \includeonly statement, which
will trigger a complete build.

More improvements to follow.

Fix correction with continuous spellcheck

This commit amends 8d26ae57ba66b4518985cf3430de2dcf37096568.

Fix UI quirk in IncludeOnly settings

Fixes #11564

outline-up/down: preserve cursor's position

There is no need to change the cursor's position to the beginning of
the line.

This change does not affect the behavior of using the buttons in the
Outliner pane, but is useful for using the keyboard shortcuts.

Fix Qt deprecation warns for setting tab

Fix warnings coming from deprecations of QTextEdit::tabStopWidth()
and QFontMetrics::width(). Regarding tabStopWidth(), the ChangeLog
states the following [1]:

  Introduced tabStopDistance property in QTextOption, QTextEdit and
  QPlainTextEdit as replacement for the inconsistently named tabStop and
  tabStopWidth properties. QTextOption::tabStop, QTextEdit::tabStopWidth and
  QPlainTextEdit::tabStopWidth are now deprecated.

Note that QFontMetrics::horizontalAdvance() is what we want here, as
opposed to QFontMetrics::boundingRect(), because we want to know
where to draw the next character after the tab.

[1] https://code.qt.io/cgit/qt/qtbase.git/tree/dist/changes-5.10.0/?h=v5.10.0

#11747 follow-up of change d94b708a0fec6ade04fc9d09bd86afd73e72fac7 to fix a compiler warning

#11767 improved default binding on Mac for window-close

Revert "Do not display caret when we're not ready"

This reverts commit e99436b529fc83a5a89e3ec053f977117c339082.

Kornel discovered a regression due to this commit. See:

  https://www.mail-archive.com/search?l=mid&q=20200307121525.56be047b%40admin1-desktop

Revert b7509419.

There are double updates of dialogs in some cases, but this isn't why.

Cmake build: Amend b399cfc7

Re-allow also the previous include, so that it is now
possible to use
#include <hunspell.hxx>
and also
#include <hunspell/hunspell.hxx>

Cmake build: Remove warnings about deprecated policy CMP0075

Consider CT underlining pref in captions markup

Fix Qt deprecation warn for QPixmapCache::find()

Fix the following warning from Qt 5.14.1:

  error: ‘static bool QPixmapCache::find(const QString&, QPixmap&)’ is deprecated: Use bool find(const QString &, QPixmap*) instead [-Werror=deprecated-declarations]

Fix Qt deprecation warn for QList::fromSet()

Fix the following warning from Qt 5.14.1:

  error: ‘static QList<T> QList<T>::fromSet(const QSet<T>&) [with T = QString]’ is deprecated: Use QList<T>(set.begin(), set.end()) instead. [-Werror=deprecated-declarations]

Regarding QList::fromSet(), the documentation now states the
following [1]:

  Since Qt 5.14, range constructors are available for Qt's generic
  container classes and should be used in place of this method.

[1] https://doc.qt.io/qt-5/qlist.html

Markup changed captions in workarea by underline and strikeout, resp.

Footnote: (re-)add a fallback dummy counter

Use real counter in deleted footnotes

Fix Qt deprecation warn for setConfirmOverwrite()

Fix the following warning:

  error: ‘void QFileDialog::setConfirmOverwrite(bool)’ is deprecated: Use setOption(DontConfirmOverwrite, !enabled) instead [-Werror=deprecated-declarations]

Note in the change that "false" changes to "true" because the option
is *Dont*ConfirmOverwrite.

QFileDialog::setConfirmOverwrite() is obsolete at least since 4.8
and QFileDialog::setOption() is available since 4.5 [1].

[1] https://doc.qt.io/archives/qt-4.8/qfiledialog-obsolete.html
[2] https://doc.qt.io/qt-5/qfiledialog.html#setOption

Fix Qt deprecation warn for getContentsMargins()

Fixes the following warning:

  error: ‘void QWidget::getContentsMargins(int*, int*, int*, int*) const’ is deprecated: use contentsMargins() [-Werror=deprecated-declarations]

The new code also has advantages in that 'l' can be set to const and
it is more readable because it is clear that only 'l' is used.

QWidget::getContentsMargins() was deprecated in 5.14.0 [1]. and
QWidget::contentsMargins() is available since 4.6 [2].

[1] https://code.qt.io/cgit/qt/qtbase.git/tree/dist/changes-5.14.0/?h=v5.14.0
[2] https://doc.qt.io/archives/qt-4.8/qlayout.html#contentsMargins

Handle deleted BibTeX/Biblatex bibliographies

Since we cannot strike-out these, we output a placeholder when
show changes in output is active.

Fixes #11762

Tell updateBuffer whether an inset is deleted.

Fixes #11747

Consider change tracking in setBeginOfBody()

Fixes #11749

Use range-based loop

Account for changes in insets when checking changes in selection

This is not ideal, see FIXME.

Fixes #11774

Fix Qt deprecation warning for isItemSelected()

These methods are related to the deprecated setItemSelected() and
setItemHidden() methods that were converted at 24926b2e.

Fix Qt deprecation warning for setItemHidden()

Consistent with commit 24926b2e.

Rename a var "sort" to not mask std::sort()

This allows us to use sort() without the "std" prefix.

Fix deprecation warnings from use of qSort()

This commit replaces qSort with std::sort to fix warnings from compiling with
Qt 5.14.1. Below is one of the warnings:

  error: ‘void qSort(RandomAccessIterator, RandomAccessIterator, LessThan) [with RandomAccessIterator = QList<lyx::ColorCode>::iterator; LessT$
  an = bool (*)(lyx::ColorCode, lyx::ColorCode)]’ is deprecated: Use std::sort [-Werror=deprecated-declarations]

qSort() has been deprecated since Qt 5.2. Quoting from the ChangeLog [1]:

  With STL no longer being optional for building and using Qt, a number of
  parts of QtAlgorithms no longer make sense, and have therefore been
  deprecated. Replacements are available in the STL, and generally have
  much better performance

There are some cases that require more than just a trivial substitution, but
our code does not appear to use any of those cases.

For some discussion on the differences in speed of std::sort() and
qSort(), see the following:

  https://phabricator.kde.org/D10857

These are just warnings now, but will likely be errors with Qt 6:

  https://bugreports.qt.io/browse/QTBUG-73048

I tested that LyX can still be built against Qt 4.8.7 with this commit.

This commit follows 24926b2e, which also fixes some deprecation warnings.

[1]
https://code.qt.io/cgit/qt/qtbase.git/tree/dist/changes-5.2.0/?h=v5.2.0

Fix a few deprecation warnings in Qt 5.14.1

These changes fix a few instances of the following type of warning:

  error: ‘void QListWidget::setItemSelected(const QListWidgetItem*, bool)’ is deprecated: Use QListWidgetItem::setSelected() instead [-Werror=deprecated-declarations]

as well as similar warnings for setItemHidden() and
setItemExpanded(). These are just warnings now, but it is planned to
remove the methods for Qt 6:

  https://bugreports.qt.io/browse/QTBUG-73048

I tested that LyX can still be built against Qt 4.8.7 with this
commit. Indeed, these methods have been deprecated for a while (it
is just that QT_DEPRECATED_WARNINGS was only turned on by default
starting with 5.13.0). See, e.g.,

  https://doc.qt.io/archives/qt-4.7/qlistwidget-obsolete.html

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.