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.

Implement '*' in "buffer-export <FORMAT> <DEST>"

For the variant of buffer-export that allows a format and a
filename, the substitution must be done after the argument is split.

Related to 494ce664.

Typo: Copy/Paste error

Cmake build: Adapt handling of mytheslib to automek behaviour

The added cmake-parameter is:
  -DLYX_EXTERNAL_MYTHES=<value>
where <value> is one of
  AUTO (Default) Search first on system for mythes (lib and include)
  ON Use installed only (errors if not installed)
  OFF Compile the provided source in 3rdparty

Fix typo found by coverity

We were not testing for the right end(), although it is not sure that
an actual bug could be triggered because of that.

Fix test to make coverity happy

It is not obvious that cur is not null iff thisSlice is not -1, but
this is what happens in the caller.

Fix test to make coverity happy

It is not obvious that cur is not null iff thisSlice is not -1, but
this is what happens in the caller.

Fix distcheck for mythes

For some reason, I mistyped "myspell" instead of "mythes" as a reflex. I
thought that I fixed all occurences, but it appears that I missed
one...

Amend 0e50ad8 'move mythes sources to 3rdparty' for cmake build.

Update bundled mythes to version 1.2.5

Move it to 3rdparty/ directory alongside the other ones.

Rewrite detection of MyThes library

Now configure searches in this order
1. system-installed library
2. code bundled with LyX

If --with-included-mythes or if no mythes library is installed, then
the script selects the bundled version. There should be no cases
leading to error message.

Amend 841825aa563

sort alphabetically

Fix color of todo notes

Yellow on blue causes display issues (some text elements that become black on
blue, math insets that are no longer readable). Switch to black on orange to
match the pdf output.

Add default export to the menu

Fix warning

Allow Ctrl+Enter in GuiCitation even if the citation is already there

Tabular::editXY should not touch the selection

buffer-export * synonymous to buffer-export

ja.po: update for master from Koji

Fix bug #10579

The \lyxdeleted macro cannot cope with empty lines.

Fixup to 2f701e6a1

Use the bundled mythes by default again, since the current situaiton
create an error for anybody who does not have the system one
installed.

This is a stop gap measure and I'll return to it later (in a hurry now).

Cmake tests: Add visual indication if a test is ignoring some latex error messages

Try to see if coverity annotations should be in lower case

The current one did not work.

Initialisize properly member variables

Instead of using a coverity annotation (that does not work) it is
better to really iinitialize the members of the object, just in case
somebody decides to actually use the DisplayPath() constructor.

Please coverity properly

Clarify configure help for 3rd party code

The configure code suggests that the default for
--with-included-(iconv|zlib|hunspell) is "yes", which is wrong: by
default is indeed to use the system libraries.

Change the MyThes detection to use the system version by default.

Also add to the verison information a line which lists the bundled libraries.

ctests: Add test file for "exotic" inputencoding cp858.

Encoding cp858 is only supported by some iconv variants
Gnu iconv only supports it, if configured with "--enable-extra-encodings"
(see https://www.gnu.org/software/libiconv/)

Maybe drop support or add a configuration check?

Remove regex which does mot mach any test

ctests: Fix some "unreliable" patterns.

Try another way to signal a false positive to coverity

Do not return a reference to a temporary variable

Coverity correctly spotted that the existing code creates a temporary
map and returns a value from it. It is not possible to make the map
const& directly because operator[] may change the map.

Therefore, we use map::find instead.

Add missing initialization

Spotted by coverity

Initialize properly TextEntry variable

Not sure it fixes a bug, but it should calm down coverity.

Initialize properly several InsetQuote members

Spotted by coverity

Fix coverity false positive

The fall through was intended at db889bc6.

Fix broken logic

Spotted by coverity.

Initialize pointer class member

Found by coverity.

Initialize class member to please coverity

Initialize class members to please coverity

Initialize properly a class member

This was forgotten at 0552563c99, and it only meant to keep coverity happy.

Update sk.po

Amend 34d2f3c.
Regexes in *Tests have to be specified for the whole test-names.

Allow Input of local includes from local layout files

When including files, LyX always searches the user and the system
directory, in that order. This means that when using local layout
files broken down into multiple includes, the various includes should
be specified with a path relative to the user layouts directory
(typically ~/.lyx/layouts), making this very impractical.
This commit allows including local files by specifying their path
as explicitly relative to the main layout file position, i.e., by
specifying their path with either "./" or "../". If the main layout
is not loaded from a local file, the usual search order is used,
even if the path are explicitly relative. So, for system layouts,
both "Input ./name.inc" and "Input name.inc" are equivalent.

Ignore missing glyphs in a test after TL update

After a TL 2016 update, a test started to fail due to missing
glyphs. For more information, see the ML thread here:

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

Invert a pdf4 test failing because of iconv

This is consistent with other tests failing for the same reason.
For more information, see #9871 and the discussion at

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

   Update fr.po

buffer-export without argument exports the default format

buffer-export is proposed as a default binding in the preferences so now it does
what a user expects when binding it to a key.

ctests: invert failing Spanish docs, tag unreliable examples.

Babel-Spanish uses utf8-autostrings if it detects Xe/LuaTeX.
This leads to wrong output and/or failures with
8bit TeX-fonts and 8-bit input encoding.

Remove LyXToolBox

It extended QToolBox with size calculation to fix UI issues with the previous
design of GuiCitation. Since it always needed some kind of hack and since
QToolBoxes do not appear so popular, it probably won't be used anymore.

Give focus to the filter in GuiRef

This is now consistent with GuiCitation. Compared to the previous behaviour
there is only one down key press difference.

Fix focus events in GuiCitation