Minor amendment to b95262f48772

Just for consistency.

Avoid duplicating mode changing commands

Ensure proper mode in mathed

A math-mode command entered in a text-mode environment is wrapped
in an ensuremath inset, as well as a text-mode command entered in a
math-mode environment is wrapped in a text inset. Who doesn't like
the extra insets can always dissolve them and rely on the wrapping
performed on export.

Test and fix lib/unicodesymbols for Latin, Greek, and Cyrillic.

Math definitions of Greek symbol variants for lib/unicodesymbols

+ test sample for Greek and Coptic Unicode block

Fix data loss with [ in first cell of aligned

If the first character in the first cell of an aligned math environment is
'[', and the environment does not use top or bottom vertical alignment,
then LyX did write the '[' unprotected so that it got misinterpreted as
optional argument, both when reading the .lyx file in LyX and when reading
the .tex file in LaTeX => data loss!
The fix is to output an empty optional argument in this case, which is
interpreted as default alignment both by LyX and LaTeX. It would also be
possible to output \[ in the first cell instead, but this would be more
difficult to implement.

Prepare for mutlicolumn lines

The \multicolumn command allows to set vertical lines for individual rows.
These are not yet displayed, but if they are supported one day, the code in
a27ff13663 needs to be adjusted. This change hints at the adjustment.

Set vertical lines in math arrays correctly on Y axis

This is a fixup to commit 39329935. The two fixes are
* add forgotten offset `y' when drawing the line
* in order to have a continuous vertical line, draw from the offset of
  the previous row.

Fixes bug #10363.

Set window title according to platform UI

The window title is built from the current file name and its
mofidication state. We use our own code instead of the automatic title
bar provided when windowFileName() is set because

1/ Qt does not keep the full path name
2/ Qt does not yield a nice application name

The "read only" and "version control" status are shown in the status bar:

* for read only we use the tab read only emblem (with the right size)
* for version control, we show the name of the backend (using a new
  vcname() method of the backend).

The iconText() of the view is not updated anymore, since this is
deprecated in Qt5.

Sanitize Ref dialog validation.

Fixes: #10376

Update docs wrt Separator

partly Revert "fr/UserGuide: remove spurious language switch in an index inset."

Please fix at first the versions in branch since this is the working copy delivered with LyX.
Also please keep the fileformat unless you need to document a new feature that requires a new fileformat.

ctests: More inversions

* Polyglossia + LuaTeX + \verb bug with Cyrillic and Greek
* PDF(LuaTeX) Intro.lyx (reports png without bounding box)

ctests: one more languagenesting sample

fr/UserGuide: remove spurious language switch in an index inset.

This led to errors when compiling with polyglossia (and non-TeX fonts).
A minimal (currently non-compiling) test sample is kept in autotests/export/
and inverted in suspiciousTests.

ctests: edit test sample for bug #10371

Fix compilation of attic/id_UserGuide.lyx

Change an ERT \bf{ to \textbf{.

Consistent with e.g. 0e49231b.

id_UserGuide: Update LyX format for next commit

ctests: "nonstandard" tests are now listed in unreliableTests

... under the sublabel "nonstandard".

minor edit of development guide

ctests: invert tests due to Polyglossia + LuaTeX + verb bug with Cyrillic

Remove duplicate context menu item

Fixes: #10368

installer: major third-party updates

Add xlsx format

We do already have docx, but xlsx was missing. This is a separate format
because of the MIME type. nd because some users might need converters which
can only handle one format. Now the spreadsheet template does not hide the
fact anymore that it can deal with xlsx files as well.

Fix filter filter regex

Now the external material file dialog shows the file types defined in the
template again. Before this fix it did only show "All files (*)".
I don't know what the old syntax was supposed to do or whether it ever worked
with boost::regex, but with std::regex the new one is correct.

Amend commit e194c9ce279

As noted in the log, this commit was a bit radical and needs some
adjustments (which was expected, actually).

Let InsetText do their own background drawing.

Fixes bug #10359.

Amend 97b9e81ceef7aae

\@ifundefined is an If-Then-ELSE construct.

French doc files: port last changes to master

Remove English traces in IPA inset that make XeTeX fail.

ctests: xy package has issue with newest LuaTeX

LuaTeX export of docs with xy package are now marked as having TeX
issues. Hopefully these will be fixed in a TL package update.

Exit with error if child exits with error (#10188)

Before, it could have been the case that

  lyx -e pdf2 file.lyx

had exit code 0 even though file.lyx includes a file that exits with
error. If compiled in the GUI a warning was given, but from the
command line exit code it would seem there was no problem. The exit
code for this case is now non-zero and the word "Warning" is now
removed from the message because it should be treated as an error.

An exception is thrown from InsetInclude and is caught in
Buffer::makeLaTeXFile() and added to the error list.

The (similar) use case at #8840 is also fixed by this commit.

ctests: ignore obsolete aastex.cls

aastex.cls is obsolete and replaced by aastex6.cls.

This is consistent with how we treat the obsolete latex8.sty.

ctests: mark failing LyXHTML tests as "lyxbugs"

For more information, see #10355.

ctests: uninvert three tests

These exports now succeed, and manual inspection of the output
confirms there are no obvious problem.

he/splash.lyx: mark \LaTeX{} insets as English

This makes sure they're output in the correct direction, consistent
with the previous commit (although this does not change the exit
code of the export).

For more details, see:
https://www.mail-archive.com/search?l=mid&q=20160828233923.guhveaqxdxh356nm%40steph

he/splash.lyx: fix compilation

We need to mark the LyX inset as English to avoid "missing glyph"
errors.

For more details, see:
https://www.mail-archive.com/search?l=mid&q=20160828233923.guhveaqxdxh356nm%40steph

ctests: update tests to reflect lualatex fixes

europeCV and modernCV examples can now be exported to PDF using
LuaTeX. For the specific output that was fixed, look at the diff and
see the description in suspiciousTests that was removed by this
commit. The output was checked manually and appears fine. These
tests are thus "uninverted".

Exporting those examples to DVI with LuaTeX does not exit with
error, but the output drops characters with accents. Thus, these
tests are now marked with the "wrong_output" label. I reported this
issue on the LuaTeX mailing list at [1], but since DVI export is not
given high priority, I don't expect much action.

Note that these changes reflect an updated TeX Live 2016
installation.

[1]
https://www.mail-archive.com/search?l=mid&q=20160831134006.4fewxothddqfeyw4%40steph

Remove unused 'glue' parameter

It was probably useful at some time, but not now.

Remove unused struct members

Interestingly, cppcheck detected that they were not initialized (see
8a047a41), but not that they were unused.

Improve status check of LFUN_BUFFER_ZOOM_{OUT,IN}

buffer-zoom-out can actually zoom in (by giving a positive number as
the argument), so it should only be disabled at the minimum zoom
level if it is indeed zooming out.

Similarly, buffer-zoom-in can zoom out, so it should be disabled if
the user is at the minimum zoom and it is called to zoom out.

This commit amends 8884c404.

Cmake build: Add format V2.2 to export tests

Amend b583fb26e89

It failed in the case where a selection already existed (the anchor would be wrongly reset).

Amend previous commit, 0e49231b

Thanks to Guillaume.

Convert deprecated '\rm' to '\textrm' in examples

This change fixes the export of our FeynmanDiagrams.lyx example
files on the latest TeX Live 2016.

The IJMP(C|D).lyx examples compiled before the change but might as
well convert these instances of '\rm' also.

The changes were made with the help of the following command:

  find ./ -iregex .*lyx -exec sed -i 's/\\rm /\\textrm /g' {} \;

and the resulting diff was confirmed manually.

This commit is consistent with 3cecd4d3.

Action.cpp: replace a reference with a shared_ptr

Replace the member reference to FuncRequest in Action.cpp with a
shared_ptr. Compared to copying the FuncRequest, the shared_ptr has two
advantages:

* Recreating the menu each time creates a lot of new actions, so we avoid a lot
  of copies.

* FuncRequest can remain forward-declared in Action.h.

Change "Format" into "Formats"

The plural form seems more appropriate and the tag [[output]] is added for
disambiguation (e.g. for the French translation).

producesOutput() should not be redeclared as producesOutput(bool).

Trivial clean-up

Fix Unicode use in Format's prettyname

The field prettyname can accept Unicode and therefore must be parsed into a
docstring.

Little simplification of the code on the way.

* For other fields, either a validator should be set to prevent non-ascii input
  in the preferences, or they should be transformed into docstring too.

Fix the compare_locale function by relying on Qt's (#9030)

Add more lib/symbols unicodemapping

There are still many math symbols in lib/symbols that lack a corresponding
entry in lib/unicodesymbols, although a clear mapping exists. This commit
adds some of them (not all yet). In the future we should probably move the
information from both files into one database.

Update it.po

Add a wrong_output ctest for mixing inTitle #10347

As of 0b1cf133 we now warn in the GUI of this issue, but there is a
discussion about whether we should change our LaTeX output and allow
for the workflow of mixing inTitle layouts. For more information,
see #10347.

Update sk.po

de.po

Implement proper dialog titles in InsetParamsDialogs

Fixes: #10329

Fix \@ifundefined check for covexample(s)

Fixes: #10340

Do not set layout for no-op paragraph break

When breaking paragraph in an empty top-level paragraph, nothing
happens on screen but yet there is an undo step because the layout is
reset to what it already was.

Avoid this case.

Fixes bug #10089.

Move correctly the end of proof marker when row is large

This was a regression from 2.1.x in LtR mode, and it never worked correctly in RtL mode.

Fixes bug #10344.

Reset anchor before selecting with S-button1

Let M-x give focus to minibuffer when it is already open

Correct library name of installed hunspell framework (it does not match the patch version number)

Remove unused variable warning and remove preprocessor switch.

Quote filenames used for forward-search with SumatraPDF

Single quotes are special to our parser and must be either escaped or
quoted if they are part of a filename, otherwise they are stripped out.

See #10342.

Fix regression of 88603655

The XHTML entries were added in the wrong field, if they are given we need
extra xhtml requires. The order is inconsistent, there are discussions on the
list about that but for now I'll use the existing order to fix the export tests.

The format of the lib/symbols file does not support "" for empty strings.
One would have to use x for fields that are not set (this is a hack), but
actually we know how to fill the extra field.

Add check for existance of Qt translations directory

Use only one file for dummy implementations

Update sk.po

Add missing entries in syntax.default

These are: \cellcolor, columncolor and rowcolor

Fix \rotatebox

Remove ",," in argument of color command, this is actually not used.

Remove horizontal spacing around previews

This concerns the preview version of InsetPreview and InsetIPA.

Fixes bug #10304.

Simplify menus when there is only one caption type

This removes the submenu indirection in Insert and the type changer in
contextual menu. Interestingly, the code was there, but it did not
work at all.

Cmake build: Add verbose variable to some dummy_functions.cpp

Warn in GUI when mixing Title and InTitle layouts

Since we process layouts sequentially, we export LaTeX code for the
title once we arrive to a layout that has InTitle false. If the
document then later has a layout with InTitle true, we do not
(currently) go back to add it to the title and just output it
in-place. We previously warned with LYXERR0, but since this can
cause missing or unexpected output we now warn in the GUI.

For more information, see the following lyx-devel thread:
https://www.mail-archive.com/search?l=mid&q=a65ae226-d3bd-8fc5-a93b-7bb23f1cda82%40lyx.org

Fix exit code from runCommand on Windows

On Windows, runCommand has never returned the exit code of the
spawned process but the result of correctly closing the input handle.

Fixes #10327

Add verbose option

Using the option -v (--verbose) all spawned commands are print
on the terminal. Note that this was done by default on previous
versions.

Use proper QPalette::NoRole enum value instead of our own

A more radical approach to inset background painting

Now by default all insets paint their own background when needed. This
means that 63cf3297 and part of 9940acc5 can be reverted.

To avoid extra painting, background drawing is disabled for
InsetCommand and InsetCollapsable. These insets draw background as
part of their normal drawing activity.

This will avoid drawing artifacts with InsetNewpage, InsetVSpace and
probably some others.

A PainterInfo variable should be named `pi'.

Assertions when doing Export As... (#10321)

Fix assertion with gcc 6: The comparison function must be a strict weak
orderings and not give x < x.

Fix assertion when a custom exportable document format is given a non-ASCII
name. Use qt_ to be consistent with the rest of the code.

Use Qt's locale-aware comparison for appropriate sorting.

Adapt code to comment

Correct comment

According to C++11 rules, static locals are thread safe for
the first-time initialization.

See also:
http://stackoverflow.com/questions/12302057/c11-safe-double-checked-locking-for-lazy-initialization-possible

Commands to select the posix variant for the mingw cross-compilation script

The win32 thread model of gcc does not support the <thread> and <mutex> headers
yet.

Foils.lyx: fix output of author

By moving date and author above the standard layouts, author is now
output in the PDF and the terminal error from LyX about mixing an
InTitle layout with non-InTitle layouts is gone.

Literate.lyx: move date with other InTitle layouts

This leads to the date being output in the PDF and gets rid of the
terminal error from LyX about mixing an InTitle layout with
non-InTitle layouts.

Improve a layout LYXERR0 message

The name of the layout is now included in an error message about
layouts. This will help with finding the root issue of several
documents we have.

Update tex2lyx tests.

Cmake build: Ignore also files with extension .patch --

These files should not be installed.

Fix dviluatex exports of Sweave/knitr/Lily/noweb

After the fix at fe06ef0e, several converter chains to DVI
(dviluatex) were broken because they had previously gone through an
incorrect link in the chain.

This commit adds converters for the above formats to dviluatex and
fixes the following ctests:

  export/examples/Literate_dvi3_texF
  export/examples/Literate_dvi3_systemF
  export/examples/knitr_dvi3_texF
  export/examples/knitr_dvi3_systemF
  export/examples/lilypond_dvi3_texF
  export/examples/lilypond_dvi3_systemF
  export/examples/noweb2lyx_dvi3_texF
  export/examples/noweb2lyx_dvi3_systemF
  export/examples/sweave_dvi3_texF
  export/examples/sweave_dvi3_systemF
  export/templates/RJournal_dvi3_texF

Cmake build: Amend 41e409f8: Use std::call_once only if it is actually available

Use std::call_once only if it is actually available

Remove debugging code.

Fix XHTML export of German Additional Features manual.

There was an oddity in the manual that exposed a problem with the
test for the "special case" of an inset all by itself in a pargraph.
If a font change is applied to that inset, we still need to open the
paragraph.

InsetNote is not output for XHTML.

Also fixes whitespace.

Debugging code.

We don't need to crash here, but can try to continue in release
mode.

Remove stray "typewriter" font.

Special characters should be output to the TOC.

Fix bug #10320.

Layout::write needs to allow for the possility that the name of the
style may contain a space. It seems sufficient just always to quote it.

Let InsetSpecialChar draw its background when necessary

This is the same fix as in 9940acc5 for auote inset and math macro template.

Partially revert "Replace static with thread_local when used for caching"

As noticed by Stephan, clang on Mac is nowhere near to support thread_local for
some reason:
http://stackoverflow.com/questions/28094794/why-does-apple-clang-disallow-c11-thread-local-when-official-clang-supports

It does not look realistic to me to provide a configure-based alternative. The
solution is to not use thread_local until it is reasonably supported on
Mac. According to sources, this means requiring Xcode >= 8.

This reverts commit 82d4f1a446260333ef1590f3622196e7c76e414d partially.