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

Amend 72a488d7

- Mention the new buffer parameter in development/FORMAT

- Assure that lyx2lyx generates the same output obtained when saving
  a converted document from within lyx

- Don't require any action when converting a document created with
  lyx 2.2 in order to assure unchanged output (thanks José for the hint)

Fix missing OutlinerName in simplecv

Fix output of en- and em-dashes with TeX fonts

This commit fixes the regression introduced in 2.2 about the
output of en- and em-dashes. In 2.2 en- and em-dashes are output as
the \textendash and \textemdash macros when using TeX fonts, causing
changed output in old documents and also bugs (for example, #10490).

Now documents produced with older versions work again as intended,
while documents produced with 2.2 can be made to produce the exact
same output by simply checking "Don't use ligatures for en-and
em-dashes" in Document->Settings->Fonts.

When exporting documents using TeX fonts to earlier versions, in order
to avoid changed output, a zero-width space character is inserted after
each en/em-dash if dash ligatures are allowed. These characters are
removed when reloading  documents with 2.3, so that they don't accumulate.

Consider text-mode accents of the form {\v a} in BiblioInfo

Fixes #9340.

repair Cursor::x_target mechanism (#10168)

Regularly check if preview is modified when visible on screen

In case one cannot rely on QFileSystemWatcher.

Regularly check if graphics is modified when visible on screen (#10596)

In case one cannot rely on QFileSystemWatcher.

see https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg199112.html

Cosmetic

Ressurect old FileMonitor à la Frankenstein

ActiveFileMonitor combines QFileSystemWatcher with the previous checksum
approach.

Blocking signals also blocks the notification of file creation (#10595)

Correct ownership

One can now delete a file monitor if a blocker is active.

Improve BibTeX name parsing #4

Handle name prefix (aka "von" part) as a separate entity, just like
BibTeX and Biblatex do. This allows to omit or reposition it in
accordance to the current style ("Goethe, Johann Wolfgang" or
"von Goethe, Johann Wolfgang" or "Goethe, Johann Wolfgang von" are all
valid and used).

LyX's name parser should now be on par with BibTeX's.

Replace "junior" with the more generic term "suffix".

Improve BibTeX name parsing #3

Correctly handle name suffix ("Jr.-part")

Allow for simple conditions in name scheme.

I.e., only output a comma between last and first name if there is
a first name.

Improve BibTeX name parsing #2

Also consider grouping when looking for name separators.

Cases such as {{Barnes and Noble, Inc.}} are now handled correctly.

Improve BibTeX name parsing #1

Consider groupings of name parts via {...}

Fix typo in comment

Use "Prename Surname" format for basic numeric styles.

Clean up

Only keep one dynamic-cast. This fixes coverity warnings.

Input stdciteformats to the cite engines directly

not to individual layouts.

Possibly fixes #10582

lyx2lyx: Remove UTF-8 BOM if present in the lyx file

Output debug message when creating bad length

Coverity is right to say that we should check the return value.

Another forgotten unreachable `return' that is not needed

The compilers we use now do not care, but coverity complains.

Fixup commit 50060053

Fix two instances where theFormats() was used instead of a local parameter.

Fix layout test.

Only accept non-negative lyxscale parameters

Since lyxscale is unsigned, a negative value would lead to a very
large positive value.

Spotted by coverity.

Move the global formats and system_formats variables into the
LyX singleton.

Mostly, this is very boring, but it might be good if someone would
check what I did about the dummy implementation in tex2lyx.

Use FileName::lastModified() intead of stat() in DepTable

This patch removes explicit calls to stat() in the DepTable code. The
original motivation was to get read of a stat() invokation where the
error condition was not checked (spotted by coverity).

Remove optimization that avoids drawing some math inset cells.

It is not clear that the optimization is a gain, but it is clear that
it has a cost: some elements of formulas will not be present in the
position cache.

Fixes bug #10520.

Make InsetMathNest::editXY more robust

If no reasonable cell has been found to put the cursor in (maybe
because the cell exist but is not in the cache), exit gracefully.

Fixes bug #10520

Do not use the global variable `formats' in Formats class

Using this global variable in the class is clearly a bug. If we wanted
to avoid doing it by mistake, the variable should be moved to the LyX
singleton.

Also add an underscore to the private variable formatlist_.

Fix problem noted at bug #10582.

When we have a name with more than two parts, but no "von",
it was coming out as, e.g.:
Obama, Barack Hussain Obama
i.e., with the last name appearing twice.

Also adds a check for names without spaces, which would have given:
Pele, Pele

This was not the original issue at #10582, so that bug is still
open (though I cannot reproduce it).

Whitespace and const'ing.

Fix compilation with 5 <= qt < 5.4

Compilation fix

It is actually better to compile code before pushing changes...

Limit the size of strings we read from RCS

This calms down Coverity which fears than an evil RCS repository would break LyX.

Update sk.po

de.po

When catching exception, give error on STDERR

The error is reported in the GUI, but for the terminal we must write
explicitly to STDERR.

For a specific example of this type of exception, see the previous
commit, 72c3dbd.

Change a warning to an error if missing include

If an included file is not found, it seems impossible that the final
output would be correct. Before this commit, when exporting on the
command line, LyX would create the PDF and exit with a zero error
code so unless the user was monitoring the terminal output, they
would not know there is a problem.

This change is consistent with 1a374a93.

Nonsense for whoever insists on using gcc4.6 & qt4.8 in 2017

Remove legacy FileMonitor

Replace FileMonitor with FileMonitor2 in RenderPreview

Replace FileMonitor with FileMonitor2 in GraphicsCacheItem

Remove dead code

Clarify dialog

Disable edition when external modifications are detected

Introduce a distinction between being read-only and having the read-only flag. A
buffer is read-only if either it has been externally modified or if it has the
read-only flag.

Notification of external modification inspired by gedit

New LFUN buffer-external-modification-clear

Clean-up

Display externally modified status

Whitespace

Implement real-time detection of external modifications

Implement FileMonitor as a wrapper for QFileSystemWatcher

The new file monitor supports both boost and qt signals. It is implemented in a
ressource-safe way.

Preventive fix inspired from ~

Amend 63be456c8

Fix crash when clicking on math previews

Add option to ignore a parameter in InsetCommandParams

It will not be saved to the file and will always be equal to a given default
value.

buffer-export default

       Update fr.po

     Class agutex.cls is obsolete: information given in the LyX template

Experiment: limit size of strings read from lib/symbols

Coverity complains that we might read strings that are arbitrary
large, and that this can be a security issue. This is a problem in
particular, when we feed these strings to from_utf8(), which coverity
flags as dangerous for some reason.

The best solution would be IMO to model from_utf8() properly, but I do
not know how to do that. Here I try a different solution, where I
cannot read a string larger than 64k from the file.

Let's see whether this removes part of coverity warnings.

Small cleanup to LayoutBox and CategorizedCombo

Coverity does not find it obvious that p is never negative. Normally
it is the case (because the items have been filtered), but it is
better to play safe.

Add label 'defaultoutput' to export tests using document default output format

Be careful about unparsable bibtex years

Handle gracefully the case where the regex that parses a year fails.

This is a fixup to ba171930 (spotted by coverity).

Another unreachable `return' that is not needed

The compilers we use now do not care, but coverity complains.

Remove unreachable return statements

They were needed long ago to avoid gcc warnings, but now all they do
is create coverity warnings.

Handle the case where doc_bv is null

It might only happens in specific cases where no file is open and for
some reason the new file cannot be opened.

Avoid dereferencing an iterator at the end

Found by coverity, could lead to a crash with badly formed kmap file.

Add a comment.

Make a test clearer

This should hopefully please coverity now. For some reason, the
annotation did not work. Should it be in lower case?

Make a false positive explicit

Pleases coverity

Fix missing initialization

This was already fixed after coverity report at bad511f3fe04aff4ff,
but this part was missing.

Initialize members spotted by coverity

Make a dummier LyXRC for tex2lyx

The old version made coverity complain about uninitialized members.
The new one is the same as in support/tests/dummy_impl.cpp.

Describe the use of '*' in -e/-E CL options

The use of '*' in buffer-export was implemented at 494ce664.