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

Cmake tests: Correct test BATCH_outline-beamer

Cmake build: Files already removed, but in 'git status' still present.

This is a try to really remove them.

whitespace

Fix Python detection on windows

Patch by Eugene Chornyi

Avoid uninitialized position variables in case of current inset not doing the assignment

Use the application id of lyx to address the running LyX instance

Put the LyX window frontmost automatically on Mac on successful reverse search

Avoid potential leak of memory pointed to by QStandardItem catItem

Handle encoding error in layout file parsing more gracefully

Issue a warning about non-decodable files, but do not break
completely.

Addresses #11736

Fix bug #11737.

We need to use the FileName methods when moving or removing a file,
so that refresh() gets called.

(cherry picked from commit d93091389896cddcd83de83dfd4ee8a9b4e9c7f1)

Revert "Fix bug #11712."

This breaks on Linux

This reverts commit fdf81a9bab2f4f50a3eaf22cc0e09ac454f470ed.

Fix bug #11712.

Fixes some escaping issues on Windows especially

Patch from Eugene.

Make cell index of tabular local for column loop.