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.

Remove another unnecessary assignment

Cmake build: Amend 1c097e56, correctly respect the patch-level

Handle also the case of creating from tar-file _and_ having 'git' installed

Remove unnecessary assignments

Make sure to not dereference a null pointer.

Properly resize columns of child document table widget

Indentation fixes

Fix a crash with uninitialized buffer member of MathData

See the details here: https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg210294.html

Allow externally provided list of components to skip for Qt framework builds on macOS

More flexible definition of Qt libraries to include as frameworks for macOS

Care for Qt library source directory rename for 5.12.x

Enhanced version string extraction for 3rd party libraries.

update canAutoOpenFile/autoOpenFile implementation to new MacOSX API, avoid deprecated calls

Fix syntax error with Python 2

Python 2 wants b"..." and gives error win rb"...".

Don't need to remove the file if it doesn't exist. (This would cause a five second delay in that case.)

Remove a finishUndo() after a recordUndo()

finishUndo() is only helpful in the presence of undos of type INSERT
or DELETE.

See the following ML thread for details:

  https://www.mail-archive.com/search?l=mid&q=b721c098-c7ee-d12f-1cdf-067dda9aa5cf%40lyx.org

Ignore geometry-related settings if the class provides geometry

Fixes #11724

Adopt a 'belt and braces' approach to bidi forcing

There are two techniques that I know of for forcing the direction of a
string, regardlessly of whether its contents is naturally LtR, RtL or
undecided.

1/ The unicode LTR/LTR override characters. This is supposed to be the
   clean way, however, it does not seem to work with Qt 5.14 (see
   #11691).

2/ The undocumented QTextLayout::setFlags method. This is used
   internally and allows to pass the (undocumented) flags
   Qt::TextForceRightToLeft and Qt::TextForceLeftToRight. This was
   used until we had issues with Qt 5.11 (see #11284).

In order to get the best of both worlds, this patch allows to enable
those two methods separately, and actually enables both at the same
time by default!

(hopefully) Fixes bug #11691.

move lyx-fedora script to attic/

Fix missing backspace in msgstr

Fix warning with --disable-nls

Amend 400706ad0525b

Cmake build: Remove cmake warnings about mismatched values of FindPackageHandleStandardArgs()

Remove dvipost from Windows installation.

Fix distcheck target

1/ The most annoying part was the error in po/. It turns out that reading
and understanding the po/Makevars file was the key. A simple change of
variable ensures that "make dist" does not try to rebuild lyx.pot.

2/ The way tests in src/ are defined meant that the
dependency-tracking files were not all deleted. This should be OK now.

Partially revert 43b81bec, export.cmake was not intended to change

Cmake config: Respect the patch-level while creating package from tar/zip-source

Cmake config: Amend 43b81bec

Typo, causing wrong package creation

Cmake config: Rename some variables to match the ones used by automake

LYX_RELEASE_VERSION renamed to LYX_RELEASE_VERSION
and got the new value of
LYX_RELEASE_PATCH
from configure.ac too

Update sk.po

de.po

Remove terminating linebreak

Make graĥics and external insets tighter

Now RenderGraphics adds offsets that depends on its parent inset.

These offsets are set to 0 for InsetGraphics and InsetExternal. A nice
consequence is that icons shown by Info inset stick out less on screen.

As an unrelated change, the "private:" specifier of these two insets
is moved to a more reasonable place.

Fixup eeb4187a: copy-paste errors

Remove some spacing above InsetColapsible when possible

Make script inset much tighter in texted.

We use the fact that offsets can be set separately for each inset.

Replace the hardcoded TEXT_TO_INSET_OFFSET by tunable values

The basic value is Inset::textOffset(BufferView*), which can in theory
change with the BufferView zoom and dpi. It is hardcoded to 4 for now.

Moreover, we introduce the virtual inset methods
(top|bottom|left|right)Offset, which can be tweaked for each inset.

No change intended (for now).

Improve comments

Cross out ct-deleted tabular rows/columns in workarea

* lib/RELEASE-NOTES

endl needs ostream header here.

Whitespace

Account for bidi (swapped columns) when calculation clines

Fixes #11717