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.

Whitespace

Document file format change

Include all [scr]article styles in beamer article layouts.

File format change. On format reversion, we need to put some extra code
in the local layout that emulates the 2.3 behavior.
Simply Input'ing the respective layouts is unfortunately not enough,
due to the insertion order.

See #9977

Correct default font of beamer article layouts

Fixes: #9977

Correct backends() allocation with japanese encoding

Fixes error introduced by ad9887da9c18.

Activate LyX window after reverse search (#10196)

"lyxclient -g" now calls the just implemented lyx-activate (see
previous commit) after server-goto-file-row. This allows the PDF
viewer to switch to LyX after executing a reverse search.

New LFUN lyx-activate to focus LyX window

On Linux and Mac OS, this action brings the LyX window into focus.
Such behavior is not allowed by Windows OS so instead the color of
the taskbar entry is changed to indicate that the window has changed
in some way.

The action is hidden in the shortcuts menu because it would make
sense to assign a shortcut to it. The only way to execute shortcut
would be if the LyX window is already activated.

lyx-activate will be used (see next commit) to allow the PDF viewer
to switch to LyX after executing a reverse search.

RELEASE-NOTES: add the new LFUN branch-invert

Fix embarrassing logic mistake

Thanks Richard for pointing at it.

de.po: fix meta data

de.po: minor fixes

Cmake build: Specify the correct value of 'PACKAGE' define in config.h

This was wrong in case where no program suffix was allowed
Spotted by Scott.

Fix problem noticed by Scott.

No need to display contents of the branch when it is open.

Thanks to Vincent for noticing.

de.po: update

Replace static with thread_local when used for caching

thread_local is a per-thread static variable, so it is thread-safe and can be
used for caching purpose.

Remove the cache for parameter lists as discussed on the list
http://mid.gmane.org/6e871431-0e87-ed2a-3e31-b63356c2391e@lyx.org.

(requires gcc >= 4.8)

Fix aliasing logic for polylines

Enable to specify the target format for converting the local layout

* New constant LYXFILE_LAYOUT_FORMAT in src/TextClass.cpp. This determines the
  layout format corresponding to the current lyx file format.

* The Local Layout pane is changed so that the "Convert" button does not convert
  to the internal layout format but to the current lyx file format, to make sure
  that the file does not become unreadable with a specified earlier version of
  LyX.

* If LYXFILE_LAYOUT_FORMAT == LAYOUT_FORMAT then LyX behaves as before. This is
  the value defined in master.

Add an option to layout2layout to convert to a particular format

Perf comments: getFormatFromFile slows LyX startup considerably

It is currently called on hundreds of files: settings, layouts, icons, cached
graphics files (incl. graphics from files that are not opened on startup).

According to callgrind, fixing the FIXME comments could speed up startup by more
than 30%.

Use call_once to ensure something is only called once

This is thread-safe.

Make static counters atomic

This ensures thread-safety

(requires: gcc >= 4.6, MSVC >= 2015)

C++11 supports thread-safe initialization of statics

A static local variable is guaranteed to be initialized only once, and in time.

Lambda expressions can be used to perform complex initialization of those static
variables on the spot.

(starting from: gcc >= 4.8, msvc >= 2015)

Fix "scroll here" feature of scrollbar (#10311)

Prevent setRange() from causing a recursive call to scrollTo(). Reduces three
calls of scrollTo() to one call for all scrolling functions of the scroll bar
(e.g. clicking on the arrow, dragging, or clicking somewhere on the scrollbar).

Basic implementation of QSignalBlocker for qt4

Fix typo in new dia converter.

Customization.lyx: port recent changes to master

Customization.lyx: accept Richards changes

Richard, please do everything that does not depend on LyX 2.3 this in the 2.2.x branch. Also don't upgrade the fileformat if not necessary.

Fix output of floats, etc, inside environments.

Fixes bug #9094.