Better implementation for corssOutLines

When trying to do implement a LaTeX feature on screen, it is always
good to see how LaTeX does it, just in case it is easy to implement on
our side.

Then we have a correct output at all DPI sizes.

Make output of check_Length.cpp more human readable.

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

#9006 care to make prefs2prefs.py python 3 compatible

Update sk.po

tex2lyx: fix the preamble output

- LyX has new preamble tags. Although that they are not relevant for LaTeX they must be created by tex2lyx nevertheless
- also fix an issue that the formula indentation was always output
- update the tex2lyx tests accordingly

update tex2lyx tests to latest file format

Length.cpp: add new unit representing \baselineskip

- fileformat change

InsetTabular.cpp: handle crossed-out text

GuiDocument.cpp: commit missing part

This should have been committed with the previous commit but failed for an unknown reason.

MathsUi.ui. move formula indentation to text layout as requested by Jmarc

MathsUi.ui: next attempt to make it look correct also under Linux

MathAtom is a unique_ptr

Fix coverity suggestion of defining a move constructor

Make runtests.py python3 ready

Tex2lyx tests with ctest depend on it

Avoid using buffer() which might throw an exception

Actually buffer_ is always valid on this codepath, but coverity is not
able to see that.

Avoid using theServer() for no reason

All it brings us is threatening to throw an exception to scare coverity off.

MathsUi.ui: adjust dimensions as requested

BufferParams.cpp; initialize formula_indentation with default value

- Buffer.cpp: formula_indentation will not always be output/written -> erase it

Row::Element::x2pos: go to nearest point also over insets

getPosNearX, which is the only user of x2pos, should always return the
nearest position.

In editXY, there is a need to return the position where the inset
stands, but it is done using checkInsetHit.

This is a simplification of commit eb4a2a19, which has been reverted
at 01f0ab64a.

Fixes part of bug #10569.

Revert "When selecting text with the mouse, inset selection happens in the middle"

This reverts commit eb4a2a190f2640d2a6ab7146cfcc347e70b57044.

editXY: make cursor positionning correct over non editable inset

Instead of using complicated (and wrong) code, it is better to use
getPosNearX here.

Also simplify the code by removing temporary variables.

Fixes part of #10569.

Set boundary correctly in getPosNearX

The code that looks whether neighbor block has a different direction
should look to the left or to the right depending on the direction of
the current block.

Fixes part of bug #10569

Require automake 1.14 and autoconf 2.65

This allows to fix distclean properly, since we know that
subdir-objects is used. When it is the case, it is not a good idea for
a Makefile to use a source file which is under the control of another
one. This could not be made to work work with old automake versions
(<1.14).

Therefore we refer directly to object files in Makefile.am instead of
source files. The actually leads to less compilation in tex2lyx.

Update the code here and there to remove compatibility hacks. Update
documentation.

Re-enable the layout test and fix the Length test, which was ovewriting
memory in test_inPixels() because lyxrc did not had a correct layout.

Make it clear that current_view_ is available when needed

In particular the LATTEST in createView tells coverity that creating a
view sets it as current_view_.

Document new behavior of "delete" LFUNs

- Describe the change in RELEASE-NOTES.
- Update the example for inset-forall.

This commit follows 71623b88.

Keep RELEASE-NOTES as ASCII

Might as well make it as simple as possible.

Remove "Bummer" from the "Save failed!" message

There is a small chance that the user could get offended by this
nonchalant word in a data loss situation.

For discussion, see

  https://www.mail-archive.com/search?l=mid&q=20170401043945.7bpmrdugtqo6dw3j%40steph

MathsUi.ui: adjust the size to fit better

Win installer: add packages necessary for Unicode math support

- and one for Arabic script languages like Urdu

support to indent formulas

- adds support for the general document class option fleqn
- adds support to specify the formula indentation
- fileformat change

Use assertions to declare things we know for sure

Doing it that way makes coverity happy.

Check both lower and upper bound for tainted loop limit

In these cases, the lower bound is not that important, but coverity insists on it.

Replace coverity annotation by an assertion

And reorganize the code a bit.

LaTeXConfig.lyx: add the noto fonts

FuncCode.h: correct a date

Add support to cross out characters

- adds support for the command \xout of the LateX package ulem
- fileformat change

latexfonts: correct previous commit

this went in accidentally

latexfonts: support for the Noto fonts

- fileformat change
- the support to scale the sf and the tt fonts can be added later, see enhancement #10611

configure.py: add support for Python 3 on Windows

fixes the last missing part of bug #9006 (hopefully)

Try another way of convincing coverity that everything is fine

We have a false positive here, but annotating with coverity[] comments
is not very convenient. Let's try to use good old assertions instead.

Win installer: add 2 fonts to packages list

tex2lyx: support for the cochineal fonts

Revert "Possible fix for bug #10295."

This reverts commit 9b0b45dc81e8f1bc5cf370f0a9a8f60b8d93f4e6.

(committed by mistake)

When converting, indicate clearly when buffer should exist

This makes coverity happy, and makes out code a tiny tiny bit better
commented.

Possible fix for bug #10295.

As it was, the comparison buffer was sharing a DocumentClass with
one of the compared buffers. I don't fully understand why this was
causing a problem, since we use a shared_ptr. But this patch creates
a new DocumentClass for the new buffer.

Fix bug #10581

Tell the math parser that we are parsing a macro definition, so that
it doesn't try to return a verbatim copy in case of failure.

Fix bug #10554

The \lyxdeleted macro cannot cope with \newpage & co.

Restrict file cache expiry time to positive values.

VSpace.cpp: fix 2 compiler warnings

A division can always lead to a non integer.

Improve drawing of math roots (\sqrt and \root)

This is preliminary work to make roots look more like in TeX output:

* correct font size for root order

* set minimum size of the root from the max ascent descent of the
  font. This is what TeX does.

* fix drawing, and in particular horizontal size.

More needs to be done, in particular:

* handle zooming and dpi correctly by removing hardcoded pixel values.

* factor the code for the two types of roots in one.

Work around bug in QTextLine::xToCursor

With RtL text, the method can be non-monotonic wrt x value (which is a
Qt bug). To work around this, we check whether positions adjacent to
those returned by xToCursor look better. Depending on whether the new
x position is too small or too large, we look backward or forward for
a better solution.

The performance is probably not great, but this is only needed for
user interactions, so the performance penalty should not be a problem.

Fixes #10569.

Model lyx_exit() to tell coverity what it does

Catch another exception

Spotted by coverity.

Play again with coverity annotation

The documentation is scarce and it is difficult to see what notation
is right.

Handle exception thrown by Buffer::emergencyWrite

Spotted by coverity

Replace coverity annotation by assertions

A better way to tell coverity that we do not care about result

When scanning CVS skip unreadable lines

Spotted by coverity.

Tell coverity that LAPPERR kills LyX too.

Followup to e10db6c7: handle InsetCitation too.

When destructing a workarea, ignore buffer exceptions

Spotted by coverity, but probably harmless.

Annotate destructors to please coverity

Coverity flags this code as not handling exception that may happen in buffer().

My own analysis is that this can never happen because isBufferValid()
does check whether buffer_ is null.

Any insght appreciated. The commit should be expeanded to more cases, actually.

Cmake build: Project names *should* be uniq

Get the file system encoding from sys instead of the locale

lyx2lyx: convert iteritems -> items to be compatible with python 2 and 3

Missing initialization in 555338048

Thanks to coverity.

Add possibility for command inset to inherit enclosing font

The RenderButton object now has this property. It is set depending on
the value of inheritFont() method that is currently only set for
InsetRef, InsetBibtex and InsetCitation.

Fixes bug #10258

Account for unicode chars in tempdir path on Windows

Limit multirow to 100 columns in mathed

Spotted by coverity, this avoids a blocking loop when parsing.

Make LFUN_REPEAT more robust by limiting to 10000 iterations

Limit the amount of derivatives in external CAS

The (arbitrary) limit is 1000.

Pointed out by coverity. A bad value could lock LyX.

Use convert instead of a stream to read LFUN_REPEAT count

At least coverity should find it safer.

Try a simpler way to tell coverity that we do not care about the result

On windows, use the unicode version of ShellExecute

When running latex one needs to have a proper buffer

Calms down coverity

Handle properly exception that can be thrown by to_local8bit

Remove the use of this function in GuiAlert.

This was spotted by coverity

Amend 16d5c49b

Remove redundant action when toggling the dashes check box.

Fix #9732: Inconsistent font properties for the preview and superscript insets

lyx2lyx: fix #9006 (python3 with non-utf8 encoding as the system default)

This patch fixes lyx2lyx running on python 3 for those systems
where the default encoding is not UTF-8, since open by default
uses the default system encoding.

lyx2lyx: remove trailing whitespaces from lyx_1_6.py

lyx2lyx: more descriptive member name (document.start -> document.initial_format)

lyx2lyx: Cosmetic clean up of the lyx_2_?.py files.

Remove the duplicated \# -*- coding: utf-8 -*-
lines that show in the third line.
As far as I know they are not used, the first line though is used
by python and text editors (emacs, vi & co) to get the file encoding.
That will also work in the second lines (because of the shebang convention)
but not in other lines.

Remove trailing whitespaces.

uk.po: translation update for master by Yuri

de.po: improve a string

Translate file filters for external material

GuiExternal: use proper model/view separation and sort the templates

de.po

Split external_templates to individual *.xtemplate files

This makes it easier to add new templates or change existing ones
in the user directory.

Fixes: #3983
The CMake changes should be double-checked.

Cmake build: Set cache variables correctly

In case of LYX_EXTERNAL_MYTHES==AUTO, the setting never made it into cache.

Fix coverity issues about exceptions

There a some exceptions related to the fact that BOOST_ASSERT throws
an unhandled exception, which is fait enough. This is handled by
uploading a modeling file to coverity.

The second batch of issues are related to the use of lexical_cast in
convert.cpp. We use now a wrapper around boost::lexical_cast that does
not throw but return empty strings instead. I am not sure actually of
when lexical_cast could fail.

Try alternative syntax for signaling false positive

Check return value of regex_match instead of looking at first match

Spotted by coverity.

Two places where coverity urges us to limit string length

These two issues make sense, but there are probably others.

Pass big parameters by address

Spotted by coverity.

Reduce horizontal spacing for simple inline equations

Linearize macros in box edit mode too.

The special mode used by macros where the macro name is displayed as
grey text in a box is now one of the possible marker types for any
inset. The new code puts the macro name below the text, which means
that only the text below the current line will be moved. This makes
edition much more comfortable. The grey border around the contents has
been removed too.

The marker_type enum has been moved from Inset to InsetMath. A
markerMargin() function returns the horizontal margin required on each
side for the marker. Only the corner-type markers MARKER and MARKER2
need some.

A new BOX_MARKER enum value has been added and is taken care of in
afterMetricsMarkers (renamed from metricsMarkerVertical) and
drawMarkers.

The InsetMath::marker() macro now takes a BufferView pointer as
argument, so that the marker type can depend on editing state.

MathRow::metrics now makes changes to the MathRow object when needed.

The specific metrics/draw code in MathMacro has been removed and
replaced by assertions. Code in addToMathRow and marker is adapted to
reflect the cases where linearization is possible.

Correct FORMAT documentation

Further amendment to 72a488d7

Rephrase positively the check box for the output of en- and em-dashes
and disable it when using non-TeX fonts. The state of the check box
is remembered, so that toggling the non-TeX fonts check box does not
cause information loss.

Fix compilation warning in DepTable.cpp

Also account for development versions