Fix reversion of beamer overprint when a nested paragraph has an argument

Small reorganization.

I am not sure I fully understand the pending_newline/unskip_newline
logic (which seems mainly related to rtl writing), so I prefer to
leave it alone, in the sense that now things go again as in 2.1
until the point where those booleans are used for producing output.
If it turns out that a spurious (and unwanted) empty line comes
from the previous code, it can be easily corrected later.

I am also reintroducing the check about a separator inset at the end
of the paragraph, because that is necessary for the plain version.

Generalize control on double blank lines.

We can now tell whether a blank line was just output and so can
avoid outputting another one.

Fix a GCC warning: comparing signed vs. unsigned

This also changes the type of an int to an ssize_t.

nRead is initialized as an ssize_t because it could
be negative. It is cast to a size_t for comparison
to the size of a vector, but only after we check
that nRead is not negative.

Code cleanup: get rid of RenderPreview::status()

This method is a proxy for LyXRC::preview that forces to forward
declare some wrapper around an enum...

Instead, two simple static methods previewMath() and previewText() are
introduced, that make the code much easier to follow.

Fix crash due to invalid pos

Fix compilation after afc34c7a

Initialize all members

Fix bug 9030 except for windows

The format entries should be sorted according to the culture selected for the
UI. This was not the case previously, resulting in unexpected sorting of small
and capital letters. This is now fixed by using the standard C function
strcoll(). Qt does only offer similar functionality in Qt5, and this is not
mature enough yet to depend on it.
Unfortunately we have a report that strcoll() does not work on MSVC, however
this partial fix is better than nothing. The MSVC issue might also be a
configuration problem, since MS claims that strcoll() is supported. This
still needs to be checked.

Remove unneeded slash

addPath() always adds a slash at the end, os got a double one before.
Qt and the OS are clever enough to understand that, but a single slash
looks more nice.

Move code where it belongs

It is declared in Dimension.h, so the implementation should be in
Dimension.cpp.

Ugly fix for bug #9102

The real problem is the encoding of latex_language: It is hardcoded to latin1,
but InsetListig uses the currently active encoding. Therefore, we cannot tell
whether any given character wil be encodable or not, and we should not prevent
non-ACII characters.
In the future, we need to make the encoding of latex_language dynamic, so that
it always represents the currently active encoding. Then, we could do the
correct check both for listings and ERT. For now, I simply disabled the
encoding check for listings, which also means that bug 9012 might occur in
other cases for listings, but this is less important than bug 9102.

Fix reversion of beamer documents.

Beamer documents do not have a "--Separator--" layout but a
"Separator" one. Also fix a thinko causing the deletion of
"\end_layout" tags in some cases.

Fix copy/paste error, and whitespace.

Prettify the output of InsetSeparator.

If a separator inset is used after an environment with NextNoIndent
set to false, a blank line was already output. So, avoid outputting
another blank line or an odd looking line with only a '%' character
(as the previous blank line provides already a visual separation).

Extend the otexstream class to also report about paragraph breaks.

The new method afterParbreak() returns true if a blank line was just
output and we are at the beginning of the next line, false otherwise.

Don't allow inserting two consecutive separators.

Fix crash when right-clicking into an inset with more paragraphs than the main text.

Fixes: #9123.

Disregard font properties changes when inserting a separator.

This allows to output a simple blank line instead of strange constructs.

Take into account that a blank line is always output before a command.

Whitespace.

Add limits to searches in convert_TeX_brace_to_Argument.

Correct my previous comment (thanks Kornel)

#8849 catch exception in enchant speller backend and avoid multiple speller lookups for missing dictionaries

Add comment

Graphics only handle inset-modify graphics (#8262)

inset-modify tabular and others are now passed on.

Fix alignment rendering of multirow in LyX (#8976)

Multirow cells now have the same alignment rendering in LyX
as in the output. The alignment of a multirow can change as
long as the column is not of fixed width. If the column is
of fixed width, the multirow is left-aligned.

When setting a multirow, the alignment is copied from
the last cell in the selection.

Adapt output to the on-screen representation.

LyX assumes that a standard paragraph following an aligned one or
a layout with NextNoIndent==false has to be indented on screen.
This means that in the latex output a blank line has to follow.
In this case there should be no problem as regards extra vertical
space and it simpler and more elegant to simply uncheck "Indent
Paragraph" in the Paragraph settings pane rather than changing
the current logic and allowing to insert a parbreak separator.

Fix an on-screen representation glitch.

If a layout has NextNoIndent set to true, the following paragraph
is not indented on screen. LyX checks the previous layout for that
style parameter to decide whether to indent or not. Of course,
what matters is the latex output and the on-screen representation
should match this output. Now, when a layout has NextNoIndent==true,
the latex output is correctly not indented, while the on-screen
representation may fail to match this output. This can occur when,
for example, a standard paragraph is nested in the previous layout,
because LyX would check the property of the nested layout instead
of the container layout. Thus, LyX should check the property of a
previous layout at the same depth for correctly deciding whether
a paragraph has to be indented or not.
See also http://www.lyx.org/trac/ticket/9055#comment:12 for an
example document where the previous scenario actually occurs.

Remove unused parameter.

Do not forget last word of paragraph in completion

With the old code, the last word of a paragraph would not be added in
the completion list. The key difference is to pass `from' instead of `pos'
to FontList::fontiterator.

Slight cleanup of the code.

Make use of otexstream also in external::writeExternal

This was not necessary when LyX was generously outputting newlines.
As it may happen that the output produced by writeExternal (the
result of an external inset) starts right at the beginning of a
line, if otexstream does not know that something is already on
the line, the iomanip-like variable 'breakln' fails to actually
break the line.

Cmake build: Show the found flag for use of c++11 features

Output a parbreak after a command.

This is what LyX was previously doing. It has no effect on vertical
spacing but, for example, sectioning commands stand out on the output.
The parbreak is not output if an environment follows or the alignment
of the current or next paragraph is changed.
Also remove some superfluous code.

Centralize substitution of python commands

The code for detecting python commands and substituting in the
correct prefix is now merged with what used to be libScriptSearch()
and is now renamed to commandPrep(). This commit does not change
any functionality and just improves organization to reduce the
chance of bugs in the future.

Centralize replacement of "$$s"

Now the replacement is done in startScript(). In addition to making
the code cleaner and more consistent, this commit fixes a bug where
"$$s" was not replaced when "latex=" was specified in the extra flags
of a converter.

Note that the temporary fix at 731b8610 is reverted with this commit.

Rename misleading parameter.

... and a typo and some whitespace.

* sk.po

Cmake build: Unify defines for spellchecker in config.h

Cmake build: Search for spell checking libraries unconditionally

New option: LYX_REQUIRE_SPELLCHECK. If set and no spellchecker found
  stop the compilation
  if not set, compile in many speckeckers as possible from the set
  of 'ASPELL', 'Hunspell', 'Enchant'

Cmake build: Use find_package_handle_standard_args() consistently

Update documented command for running tex2lyx tests.

Fix lyx2lyx warning for tex2lyx files

I would not have thought that reLyX was considered, but not tex2lyx!

Update version

Update tex2lyx test references

This is needed after c668ebf61. tex2lyx itself was already updated, and I
verified that the new output is correct.

Install lyx2lyx/lyx_2_2.py.

ctests: reinvert a test

This test was inverted before but got lost
in the transition to regular expressions.

Fix bugs #8546 and #9055, and introduce new separator inset.

The algorithm used for breaking a paragraph in LaTeX export is changed
for avoiding spurious blank lines causing too much vertical space.
This change is tied to the introduction of a new inset (with two
different specializations) helping in either outputing LaTeX paragraph
breaks or separating environments in LyX. Both of the above goals were
previously achieved by the ---Separator--- layout and can now be
accomplished by the new inset in a more natural way. As an example,
after leaving an environment by hitting the Return key for two times,
a third return automatically inserts a parbreak inset, which is
equivalent to the old separator layout, i.e., it also introduces a
blank line in the output. If this blank line is not wanted, the
parbreak separator can be changed to a plain separator by a right
click of the mouse. Of course, an environment can still be separated
by the following one by using the Alt+P+Return shortcut (or the
corresponding menu key), but now the plain separator inset is used
instead of the old separator layout, such that no blank line occurs in
the LaTeX output.

Old documents are converted such that the LaTeX output remains unchanged.
As a result of this conversion, the old separator layout is replaced by
the new parbreak inset, which may also appear in places where the old
algorithm was introducing blank lines while the new one is not.
Note that not all blank lines were actually affecting the LaTeX output,
because a blank line is simply ignored by the TeX engine when it occurs
in the so called "vertical mode" (e.g., after an alignment environment).
The old ---Separator--- layout is now gone and old layout files using it
are also automatically converted.

Round trip conversions between old and new format should leave a document
unchanged. This means that the new behavior about paragraph breaking is
not "carried back" to the old format. Indeed, this would need introducing
special LaTeX commands in ERT that would accumulate in roundtrip
conversions, horribly cluttering the document. So, when converting a
modified document to old formats, the LaTeX output may slightly differ in
vertical spacing if the document is processed by an old version of LyX.
In other words, forward compatibility is guaranteed, but not backwards.

Cmake build: Install lyx*.desktop and lyx*.svg files in appropriate locations for unix

Fix bug #9112: There is a test here that seems intended to catch the
case where there are unbalanced braces, but it comes too late. In that
case, we try to check cmd[docstring::npos] and crash.

Beamer block conversion: consider ERT arguments with leading/trailing blanks

Should also go to branch.

Monolingualization

Fix problem with drawing of math backround with selections.

Listings: Validate skip amounts correctly.

Fixes: #7373.

Rely on Qt to handle small caps text

This has the advantage of simplifying our code and to produce the
correct output: the small capitals should have the exact same width as
the lower case letters.

The slanted fonts are also translated to oblique on Qt side, but this
does not seems to have an effect in my testing. It may be that proper
oblique fonts need to be installed.

Simplification

Store InsetListingsParams in a vector of pairs, not a map, since order of insertion matters.

Fixes: #8144, #5203.

Fix typos in InsetListingsParams validator

make build: Versioning of lyx.svg (missed in previous commit)

Cmake build: Add also reference to versioned lyx.svg to the resources file

Turn an error message into a Debug::LOCALE warning

Cleanup the code for boost configuration

Fix View Source size

Fixes: #9084.

improve language flag for Objective-C compiler call

struct with virtual functions should have virtual destructor

Whitespace fixes

Until there is some more general solution… prepare 2.1 prefs migration manually.

Fix C++11 test

Thanks Vincent for noticing. The old test was slightly off, the C++98 standard
guarantees that __cplusplus is defined as 199711L, not 19971L.

boost, cmake: Turn off warning that signals are deprecated

boost: Turn off warning that signals are deprecated

boost: Update to 1.55.0

lyx2lyx: Update the emergency value

Actually, this should not be hardcoded.

GuiLog: Use GuiClipboard to put the log on the clipboard

All direct interaction with the Qt clipboard is now done in either GuiClipboard or GuiSelection.

Use the cached value GuiSelection::selection_supported_

That's where it is for.

Capitalize layout tag

zh_CN.po: port today's change

French doc files: backport recent changes

Merge branch 'master' of git.lyx.org:lyx

CREDITS: add Min Ding and Roy Xia

Do not write an empty default_otf_view_format.

Fixes: #9100.

Fix table rotate conversion to 2.0 format

The old code did create invalid documents, since the space between the rotate
flag dn the previous flag was removed.

README.localization: a precision

installer: fine-tuning

don't bother the user with questions if the uninstaller was accidentally deleted

Fix conversion of beamer block titles that end with a non-ERT inset.

Should also go to branch!

Fix failure to detect quoted flex insets in conversion to layout
48. Also, simplify this code.

Fix lyx2lyx math package defaults (bug #9069)

lyx2lyx sets all new math packages to off when converting from old formats.
This is correct if any command which would cause an automatic package loading
exists in the document. However, it is wrong if no command exists: This leads
to problems if later a command is added (bug #9069), which is especially
annoying for templates.
The fix consists of two parts:
1) convert_use_package() considers now the used commands like
   revert_use_package(), and uses them to decide whether to set the package to
   auto or off.
2) convert_undertilde() and revert_undertilde() use a slightly adjusted copy of
   convert_use_package() and revert_use_package(), so that the bug is also
   fixed for undertilde. We cannot use the latter functions directly, because
   of "\usepackage undertilde" vs. "\use_undertilde".

Fix math package conversion to 2.0 format

When doing the lyx2lyx round trip of the 2.0.8 user guide from format
413 -> 474 -> 413 you get an invalid document, because the math packages
mhchem and undertilde are incorrectly converted.
It turned out that the old reversion code did only work for particular
package oderings. The new one works for abitrary sorted packages.

Fix data loss in user guide conversion to 2.0.x

When doing the lyx2lyx round trip of the 2.0.8 user guide from format
413 -> 474 -> 413 you do not get a zero diff. The most important problem is
caused by the conversion of the argument insets to the old syntax: This
conversion adds an additional empty line (harmless), and it destroys the
document structure if the first inset in e.g. a subsection is not an argument
inset, but e.g. an index or label inset.
The fix is quite easy: Ensure that the paragraph begin is set to the first
argument inset.

Cmake tests: Added more regular expressions 2

Fix warning in lyx2lyx 2.0<->2.1 round trip

revert_justification() issues a warning if the \justification parameter does
not exist, and LyX itself always writes it unconditionally as well, so add it
also in lyx2lyx when converting old documents.
Currently this does not have any user visible effect, but it decouples the
default value of BufferParams::justification from the conversion of old
documents: Now it is possible to set the default to false in LyX, and old
documents will still be converted correctly.

Fix for clang detection in cmake

ctests: regex to revert two instant_preview tests

These effectively revert the following:

    export/examples/instant_preview_dvi3_systemF
    export/examples/instant_preview_pdf2

Update it.po

Cmake tests: Added more regular expressions

Make iparserdocstream more like std::istream

In C++98 std::istream does not use an operator bool(), but an operator
void*() instead, which prevents some unwanted conversions (this is one
possible implementation of the safe bool idiom).
In C++11 std::istream uses explicit operator bool, which prevents the unwanted
conversions using a new language feature.

This change does not have any effect on correct code, but prevents some
mistakes.

Do not spellcheck "code" in logical markup module

lyx_pot.py: Do not strip blanks in HelpText.

This fixes a thinko introduced in [c5ef7cfc/lyxgit]

Fixes: #9054.

cmake: Objective-C compilation support with clang and C++11

This fixes an issue when compiling with C++11 flags on: the
objective-C clang compiler produces an error. This patch set
the -std option to ansi for the Objective-C sources.

Revert "Objective-C compililation support with cmake and C++11"

This reverts commit c2e67a41ce4b3b281f6c8a805be5e821ce0aea0a.

Bibtopic requires styles files to be passed via argument

Fixes: #9060