Improve log scanner to correctly report error

When scanning the LaTeX log, previously we only looked ahead 10 lines
after a "!" line and if we did not find a line number we did not count
an error. This lead to the problem that templates/ACM-sigplan.lyx was
showing a successful export and the PDF was shown (it is still
created despite the error). Now that the exit code of the latex
command is checked (as of the previous commit), an error is correctly
given, but by parsing the log better with this commit, a more
informative error is given.

Increasing the look-ahead to 15 lines leads to correct parsing of
the ACM-sigplan log. The excerpt in the log file where there are more
than 10 lines in-between the "!" line and the line number is below:

! Undefined control sequence.
\@toappear ...ent http://dx.doi.org/10.1145/\@doi

<argument> ...n is removed.]\par \else \@toappear
                                                  \fi \if \@reprint
\noinden...

\@begin@tempboxa ...mpboxa #1{\color@begingroup #2
                                                  \color@endgroup }\def
\wid...

\@iiiparbox ...tempdima \@parboxrestore #5\@@par }
                                                  \ifx \relax #2\else
\setle...

\@copyrightspace ...planconf@finalpage}.\par \fi }
                                                  }\end@float
\maketitle ... \@copyrightwanted \@copyrightspace
                                                  \fi
l.34 \maketitle

Another example is posted here:
http://tex.stackexchange.com/questions/231655/lyx-cannot-output-to-pdflatex-for-a-specific-file

Check exit code of LaTeX process in LaTeX::run

Systemcall::startscript returns the exit code of the LaTeX command
that is run, but the return value was not being checked by
LaTeX::run.  Instead, we relied on parsing log files. However, this
parsing is not perfect.

The return value is now checked and if the exit code of the command
is non-zero, an enum value is added to the return and the user is
notified of the error.

At a higher level, if the LaTeX command returns a non-zero exit code,
in the GUI a message such as
"Error while exporting format: PDF (LuaTeX)" will be given instead of
"Successful preview of format: PDF (LuaTeX)".

When run on the commandline, lyx -e lualatex example.lyx
will give "Error: LaTeX failed" and a non-zero exit code
where before it gave a zero exit code.

A real example of the bug this commit fixes is LyX's (as of this commit)
ACM-sigplan.lyx template.
Before this commit:
  $ lyx -e pdf2 ACM-sigplan.lyx
  [...snip...]
  support/Systemcall.cpp (288): Systemcall: 'pdflatex  "ACM-sigplan.tex"'
  finished with exit code 1
  $ echo $?
  0
Starting with this commit:
  $ mylyx master -e pdf2 ACM-sigplan.lyx
  support/Systemcall.cpp (288): Systemcall: 'pdflatex  "ACM-sigplan.tex"'
  finished with exit code 1
  Error: LaTeX failed
  ----------------------------------------
  LaTeX did not run successfully. The command that was run exited with
  error.
  $ echo $?
  1

Allow cloned buffers to give alerts in runLaTeX()

This was disabled in 2009 in bea0925f but it is now safe.

From Richard:
"I am pretty sure that all the work that Peter Kümmel did with the
InGuiThread classes dealt with this issue."

Remove an outdated comment

LaTeX::deleteFilesOnError is no longer a dummy function.

This comment was interoduced with the first version of this
function, at e6d063c4.

Remove output file in temporary folder on error

This commit solves two issues:

(1) A PDF from a previous run could have been the result of a command
that exited with error (e.g. sometimes pdflatex still produces a PDF if
it exits with error). If the "View" button were clicked a second time
without changing the .lyx file, then the checksum of the .tex file would
not have changed so LyX would show the PDF (which was created from the
first run that exited with error), and this time LyX would not report
the error (because the parsing of the logs only happens when the .tex
file is compiled).

(2) A myfile.tex that results in no output does not yield a myfile.pdf.
Thus, Any myfile.pdf in the temporary directory will not be overwritten.
Before this commit, the following scenario was possible: LyX runs
pdflatex which processes myfile.tex and no error is given so LyX opens
myfile.pdf. However, it could have been the scenario that pdflatex did
not exit with error and did not create myfile.pdf, in which case
whichever myfile.pdf is being shown is not correct. To see this bug in
action, start a new document, type "abc", view the PDF, delete "abc",
view the PDF (this correctly gives an error that empty output was
created), view the PDF again (this does not give an error because the
checksum has not changed). The PDF shown will contain "abc".

Note that the above also applies to DVI files and that the fix is
general.

Clarify message about an empty file

When empty output is generated, no PDF is produced. The previous
message made it seem like an empty PDF was produced.

SVG: text -> Path

Improve source code documentation

Adjust comments to current code and extend them a bit, also revert a part
of af5acb6d which was certainly not wanted.

whitespace

Fix math macro plain text output

Previously it was not thread safe, and the translation could be wrong.

Revise svg icons for the math toolbar

- Use paths, not fonts. Have a look at the sqrt icon in the image attached
  here http://thread.gmane.org/gmane.editors.lyx.devel/154179/focus=154183
  to see what may happen when a font is not present on a given system.
- Make uniform the height of the parentheses.
- Move a couple of icons to the oxygen subdir (I was not sure it was Ok
  replacing them. If yes, they can be simply deleted).

Use integer arithmetic for drawing logos; fix LaTeX output

Using fraction of points is a reasonable way to avoid warnings with fussy compilers.

Moreover, this fixes the output of PHRASE_LATEX2E to use the proper form \LaTeXe.

The file format still uses the questionnable form \\LaTeX2e.

Cmake build: Add .svgz files to Resources.

It does not seam to make a difference.
Added because other icons are also included.

New test for seach and replace of a logo

Make tests for QT5 and QT4 work.

Changing *-in.txt due to erroneous shortcuts in it.po
keytest.py, single-test.cmake and CMakeLists.txt now
provide a frontend hint for xvkbd.

Toggle properly toolbar icons that set layout of a paragraph.

Fixes bug #9463.

Fix setting Interlingua as GUI language

The problem came from the fact that Qt associates Interlingua to locale C.
Now, we do not rely anymore on the Qt locale to set the language. I suspect that the problem will still exist if the system locale is ia_IA.

Fixes bug #9267.

fixltx2e must be loaded as early as possible

Fixes: #9452, #9361
This reverts the 'fix' to #7233, which must be solved differently.

add "Keywords" to lyx.desktop file

Fixes bug #9414.

Update it.po

Make the missing class warning opt-out

Properly handle the bicaption package (bug #9449)

Add proper importers for Sweave and knitr.

Fixes bug #8734.

Improve undo of consecutive insertions/deletions

The old scheme was:
 * multiple insertions are undone by groups of 20
 * multiple deletions are undone in one big block

The new scheme is to stop merging undo elements after 2 seconds of elapsed time.

Moreover, the merging of undo elements stops when the cursor has moved. Potentially, this could allow to remove many of the finishUndo() calls.

Fixes bug #9204.

Add missing breaks (and time for a break :-()

More switches and whitespace fixes

de.po

Grr...

Properly escape backslash in string

InsetPhantom::latex(): use a switch

Whitespace

Time to properly set up QtCreator...

Phantom is fragile (#9456)

update sk.po

Use German localization in German europass example

de.po

Refine fix for #9453

It is better to introduce a dummy blank dir in TEXINPUTS rather than
appending a blank at the end. Even if I have checked that this is not
a problem with MikTeX, some other engine (maybe texlive, but I cannot
check) could not ignore this space and take it as the name of a dir.
In this case, TEXINPUTS would not end with an empty element and the
standard search path would not be inserted there.

Add Stefan Swerk to the Credits

Add support for the europasscv class

Patch by Stefan Swerk

Fix bug #9453

This was due to a problem with the QProcess parser.
See #9453 for details.

make build: Make it compilable for Qt4 too.

Document the drawing of logos

Also fix a couple of spacings.

Improve code readability

Using a small helper struct instead of a std::pair makes things like
it->second.first a little less confusing: it->second.scope.

Add missing check

The other insert method has this as well, and it is indeed a problem is pos
points outside the vector.

Use QLocale::toDouble instead of QString::toDouble in the length validator

Fixes: #9214

Set the logic straight

Allow to disable inputenc loading via 'Provides inputenc 1'

Better oxygen citation icon

Oxygen svg: transform more text to paths

Install the svg icons.

Add last missing svg icons.

Also correctly rename an oxygen icon.

Better oxygen nomencl icon

Improve display of the logos.

This look now more like the real logos. Moreover, we use not the normal font color because Color_special is a bit too flashy.

Fix the oxygen math svg icon issues pointed out by Enrico

Cmake build: Remove deleted svg subdir from installation.

Also added missing images/math/oxygem icons to installation.

Restore some inadvertently overwritten oxygen icons.

I did not notice that all icons in math/ are shared and in a0895046 I have
overwritten some of them. I am restoring them in the proper subdir.
Two of them, namely font.svgz and frac.svgz, give warnings that should
be addressed. Anyway, I see that these icons use fonts. They are better
substitued with paths (a menu entry in inkscape) because if the fonts
are missing on a system, the icons look different and, in general, they
may look ugly.

A simplistic svg banner

This is just a proof of concept. We should find somebody who can re-draw our banner (or a new one) with inkscape.

de.po

update sk.po

Further oxygen svg fixes

Squash last icon warnings and improve rendering at default size.

Some more oxygen svg icon fixes

With this, all console errors when loading the oxygen set should be resolved

Fix rendering issue in oxygen tabular-feature_delete-column icon

Fix rendering issue in oxygen promote icon

Changed ti utf-8 encoding, otherwise python3 chokes on undecodable 0xfc character

Fix rendering issue in oxygen xref icon

Fix rendering glitch in oxygen thesaurus svg icon

Fix stmaryrd operators with limits (bug 9458)

LyX did not display the limits of the big math operators defined by
stmaryrd.sty correctly. The reason for this was a missing check in
InsetMathSymbol::metrics(), where it is hardcoded which symbols use display
style limits and which symbols use inline limits. In an ideal world this
information would be contained explicitly in lib/symbols.

This should go to branch as well.

Make sure that a generated temporary filename doesn't end with a dot.

This may happen if the extension is empty, and a filename ending with
a dot may give troubles on Windows.

Make sure an unzipped file has correct extension.

If a mask is missing, the TempFile class appends it to the filename.
This may be a problem with applications relying on the extension,
so explicitly add a mask.

Cmake build: QtGui now needed also for filetools.cpp

Move svg icons to proper place.

You have to manually delete the lib/images/svg directory as git
tracks only files.

There may be warnings on the terminal due to some faulty icons that have
to be found and corrected. Moreover, some of the icons are not rendered
too well at the default size. These ones should be properly tweaked.
Kudos to Jürgen for putting together such a large number of icons.

Prefer svg icons.

If a compressed svg icon is present, load it instead of a png one.
Also introduce two more sizes (huge and giant icons) that should be
useful when using hires displays, as svg icons automatically scale
to the desired size without loss of quality.

Use the nominal theme's icon size for displaying the icon in an info inset.

Properly name an uncompressed sgvz file.

Some tools may rely on the extension and do not actually check whether
a svg file is compressed or not.

Make sure a temporary file is always created in the global temporary dir.

Fix tex2lyx script output

tex2lyx did not use the InsetLayout for the script insets, so it did not know
that it had to use Plain Layout. However, there is still a possible pitfall:
InsetScript uses a hard coded plain layout for the InsetText constructor, so
if anybody would remove ForcePlain 1 and MultiPar false from the InsetLayout
then LyX would still write script insets with plain layout, but tex2lyx would
output standard layout again.

whitespace

Update tex2lyx test references

tex2lyx outputs the correct layout ("Plain Layout") for captions since
9d824a04d11. Previously, it did output  "Standard", which was wrong.

Preparation to support translations with python3

Thanks to José.

Avoid crash with after a params undo.

When doing a non-atomic undo, one should not try to merge with a previous params undo.

This fixes bug #7740.

First step to make po python files to support both python 2 and python 3 (3.3+)

Most of the changes are related with dictionaries returning iterators instead of
    lists. (Iterators and not views as I wrote in the previous commit message)

    xrange -> range (since xrange is gone in python 3)

First step to make lyx2lyx code support both python 2 and python 3 (3.3+)

Most of the changes are related with dictionaries returning views instead of
lists.

xrange -> range (since xrange is gone in python 3)

The code that is special to support both python 2 and 3 is enclosed in a comment
 \# Provide support for both python 2 and 3
and
 \# End of code to support for both python 2 and 3

And so later it can be removed safely when python 2 is no longer supported.

ctests: revert a MultilingualCaptions XeTeX test

This file and format exports correctly since 5753760b.

Remove unsupported macros from autocompletion

We have some math macros that exist only because LyX can display them easily,
but which require user preamble code. These commands should not appear in
autocompletion, they are only there to make the formulas of users who actually
need thgese symbols and know what to put into the preamble more beautiful.

Fix generation of lyx_commit_hash.h

Now version.o is not recompiled on already built tree.

Properly define MultiPar status of caption in the layout definition.

Also remove hardcoded paragraph break disabling.

Candidate for branch.

Cmake preparation to support python3

Take in account the parent's getStatus in macro templates

This avoids invoking the insert space dialog instead of the math version. Thereafter, spaces are correctly inserted inside macro templates.

This is the last part of the fix to #9432.

Do not sanitize the anchor of cursor when selection is not set

Additionally, move the code to write to a stream from Cursor to CursorData (so that debugging undo is easier). We loose x_target, but I am not sure it is important.

This is the second part of bug #9432.

Check that DocIterator::sanitize only adds editable insets

This fixes the crash on ticket #9432, but the bug there has other causes.

First step to support python 2(.7) and 3(.3+)

When the work is finished the moniker "work in progress" should be removed.

Forgotten in previous commit

This is not needed bacause the command 'grep' is not
so language sensitive as pcregrep. It's only for the sake of consistence

Remove the need of specific language packs for keytests

The command pcregrep failed trying to set locales to
not installed language pack.

Reduce size of some svg icons.

The svg files produced by vector editors may end up containing a lot
of unused elements that increase their size without any impact on
quality. For example, this occurs when copy/pasting between images
such that unused effects or shadings simply clutter the files.
Removing these elements in some cases may reduce the size to one half
or one third of the original one, leaving unmodified the image quality.

Add another bunch of svg icons.

Also tweaked some existing icons for better rendering at default size.

pdfcomment.module: fix dependencies

MultilingualCaptions.lyx: port recent changes to master

French MultilingualCaptions.lyx: this time really the french version

Some code factorization for the Change template

The original motivation is to initialize the change_ member but do it only once.

This fixes coverity issue 23432.

Remove variable properly now

This is a follow-up to db1979f39871.