error is always true at this point

Coverity issue 23294.

Fix unreachable code

Coverity issues 23292 and 23297.

Do not use -Wfloat-conversion with recent gcc versions

Cmake build: Adapt use of c++11 for cygwin too.

Change done according to commit 864a4db26bfa7ec05de936e9446d29bf8e58fc58

On cygwin, use -std=gnu++11 instead of -std=c++11

This allows compiling without monkeying with compiler guards.

Make it compile on cygwin in c++11 mode

When using -std=c++11, the cygwin compiler automatically defines
__STRICT_ANSI__ which is used as a guard for not declaring essential
unix standard calls such as setenv, popen, etc. As a result,
compilation stops with errors such as "xxxx has not been declared".
By undefining the guard, compilation succeeds and lyx works Ok.

Take into account symbolic links

In order to adjust the path of an included file, when deciding
whether the document was moved or not, it is not sufficient
comparing the paths because the document could have been accessed
through a symbolic link.

Use QFontMetrics information for underlines (and friends) width and position

The width of the line is now dependent on the font size.

This new computation is applied to: underline, strikeout, double underline.

Use C++11 by default when possible

Compiler that are known to support C++11 (gcc >= 4.3 and clang) are
now used in this mode by default. It is still possible to override
this choice using --(en|dis)able-cxx11.

Moreover, c++11 mode is detected from the compiler itself, not from
the use of --enable-cxx11. This allows to support compilers other
than gcc or clang.

Update INSTALL file accordingly and clean it a little bit.

Work around crash on command line export

This was a regression of 0c093a626472c7. The crash was found by the tex2lyx
test case test-insets.tex: Running lyx -f main -e pdflatex test-insets.lyx.lyx
did crash. This is not a real fix, but will do for now since previews are not
needed in command line mode (I wonder why they are generated at all?) In the
long term, we need theApp() to return a console application in commandline
mode to eliminate the existing sifferences between export from GUI and
commandline.

Remove unneeded second slash

docdir does always end with a slash, so the second one is not needed.
The first one is however mandatory.

Cmake build: Adapt detection of C++11 mode to automake

Improve detection of C++11 mode

This replaces commit 329eae56 with a better solution. Indeed, while
__cpluplus is useless with g++ 4.[3-6] because its value is always 1,
these compilers define __GXX_EXPERIMENTAL_CXX0X__ when "-std c++0x" is used.

Therefore the code now relies on both macros to detect C++11 mode
instead of setting it when --enable-cxx11 is used.

Also, use pure c++ mode instead of gnu++ extensions on gcc

Fix parsing of class and category declaration line in layout files

* Omit commented-out lines
* Properly escape backslash
* Do not allow non-space chars after delaration
* Allow blanks before # comment character

Fixes: #9746

Revert "Typo."

This reverts commit b833a32420921df529050f8301c6ca3938611ff0.

Typo.

Remove problematic workaround for "exotic" problem

Greek and Times under MikTeX with auto-install may fail due to a half-installed
font package. However, the workaround in LyX stands in the way of
alternative approaches (see bug #6469).

Add LICR replacement for COMBINING MACRON BELOW.

Add note about missing characters now errors with Xe/LuaTeX.

(cf. ticket #9610)

New lfun "accent-perispomeni".

This function inserts a COMBINING GREEK PERISPOMENI character that is
normalized to pre-composed characters for base characters where a
corresponding WITH PERISPOMENI character exists.

This is a partial solution for Ticket #6463.

Fix continuous spell checking

The extra check became useless after 4796e6b3, but its presence only
broke continuous spell checking after f65f3adb.

Fix compilation with qt5 and c++11 combined.

Fix some row breaking problems with large insets

When a row is too large due to a wide inset, it does not make sense to
break text before if the problem is the same in the next row.
Therefore give up breaking in this case.

Note that this was explicitely taken care of in the old-world
rowBreakPoint code.

Fixes bug #9691.

Fix bug #6466

\{ and \% now correctly use the selection as their argument instead of
erasing it.

Add Guillaume Munch to credits

Improve math-mode lfun for switching to text

math-mode inserts \text instead of \mbox when already in math mode,
since this supports accented characters.

Add \ensuremath to the list of math completions

Refresh file cache before checking existence

This is done only inside LFUN_BUFFER_VIEW_CACHE so as to preserve
the benefits of caching in other places.

Without this commit, if a converter or a user (re)moves the preview
file and then calls buffer-view-cache LyX will launch the viewer for
a file that does not exist.

For a discussion and use case, see:
https://www.mail-archive.com/search?l=mid&q=55D393D7.6050403@lyx.org

Fix shortcuts for copy and paste.

Quote argument of tag when necessary

Both \origin and \textclass tags may be paths and contain spaces.
In this case, enclose them in double quotes such that they can be
correctly read by the lexer.

unicodesymbols: "textgreek" must not be used together with "combining".

With the current code, this leads to output like "\\\\textgreek{\\\\~}"
which results in a LaTeX error as the accent macro \\\\~ expects an argument.

installer: fix 2 Python related bugs

see the changelog for the fixed bugs

de.po: resolve shortcut conflict

Customization: add a hint about PackageOptions at Requires.

Update sk.po

Set font in BufferView::setCursor (#9500)

This solves a few bugs related to the font not being set correctly.
For example, when putting a selection somewhere with
putSelectionAt(), the font was not reset so that before this commit
if the cursor was in an ERT, strange things could happen.
putSelectionAt() is notably used when highlighting the location
corresponding with a LaTeX error (GuiErrorList), when using find,
and when using the spellcheck. I could reproduce the bug using all
three of these.

Bug #9500 is an example of the type of bugs that this commit fixes.

de.po

Decrease height of citation dialog by using a QToolBox

Fixes: #9618

lyxpak.py: Use `subprocess` for spawning scripts.

`os.popen` is deprecated since Python-2.6. Also, this fixes the handling
of files with quotes in their names. The patch requires Python >= 2.7.

Furthermore, the patch executes the lyx2lyx script with the same
interpreter used for it, to maintain compatibility.

I also removed some lines related to Python-2.4, as it is no longer
supported anyway.

nb layout translations reviewed by Helge.

Do not attempt to select an empty cell before inserting a longtable caption

Fixes: #9692

Resolve shortcut conflict

correct symlink creation - make them relative instead of absolute
follow up of f45202acde6a98d9338f162f8ba917b2432580aa

Avoid duplicates of dictionaries and thesauri in distributed package

German genitive of LyX is LyX' (not LyXs)

(see Amtl. Regel. § 96 (1))

Remove ugly and now redundant ERT-braces from the docs

These were used to prevent the former auto-logoification of LyX etc.

Also fixed some spelling errors I passed by.

Another go at improving row shortening

Instead of remembering the last breakable element in the row, search
backward in the row and consider all string elements one by one until
a correct place to break is found.

This fixes the case where the row ends with a string, but there is no
space soon enough in the string to break it here. In this case, we
need to consider the previous row.

Comment.

Fix bug noticed by Pavel. Apparently, some changes Abdel made to preview loading for 2.1.x broke LyXHTML preview of math as images.

We have been here before. The png files are generated, but then by the time we preview the document, they are gone. This is because (a) when the PreviewLoader for a Buffer is deleted, it removes all its previews and (b) we export in a clone, so that Buffer gets deleted before the preview is actually shown. So we need to check whether we are a preview for a clone before we delete anything.

Remove ugly and now redundant ERT-braces from the German docs

These were used to prevent the former auto-logoification of LyX etc.

Also fixed some spelling errors I passed by.

preserve symlinks on copying of user directory from previous version

Do not break row at inset boundary if possible

The first change is to concentrate on the last ro elements that contain separators and try to break it a a width that is shorter than both
- its current width (we want to break it, after all)
- the amount of space available.

This simple heuristic seems to give good results.

Improved symlink creation for Mac OS for first LyX start

Use the safer addName helper function to add the configure lock name to user support directory path name

Fix bug #9684: Update previews after background color change

Merge branch 'rowpainter2'

Remember the cursor before an undo group

With this patch, the cursor position before undo can be saved when
creating an undo group. Any such value will supercede parameters
passed to recordUndo. Only the first such cursor set by nested
beginUndoGoup takes effect.

Fixes bug #9663.

Reset properly the cursor font when leaving an inset to the left

There was a missing setCursorFont there.

Fixes bug #6032.

Initialize correctly the default keymap

If keymaps are activated and there is a secondary keymap but no
primary one, then the pointer to the active keymap was 0x0. This can't
be good.

Fixes bug #9685.

Get rid of SEPARATOR row element type

It is not used anymore.

Also get rid of a comment made irrelevant by recent changes.

Reset cursor font when entring an inset with char-forward/backward

When entering an inset from the keyboard, setCurrentFont()
was not called and thus the cursor retained the font that was set
before. This could create strange behavior that could often go
unnoticed by the user.

It is easy to imagine many other situations similar to #9597 where
the user could trigger this bug.

Fixes bug #9597.

Only compute string length every 30 characters

This makes paragraph rebreaking muh much faster, at least on my ancient iMac Core 2 Duo.

Re-implement text justification

* GuiFontMetrics::pos2x, x2pos: add support for inter-word spacing.
* GuiPainter::text: idem

* Row::Element::countSeparators:
  Row::countSeparators: new methods that count spaces in strings.
  Row::setSeparatorExtraWidth: new method (code lifted from TextMetrics.cpp).

* TextMetrics::computeRowMetrics: rely on the above methods.

* RowPainter::paintMispelledMarked: pass only a Row::Element object reference
  RowPainter::paintStringAndSel: idem; do not rely on values returned by
      Painter::text (trailing spaces do not honor wordspacing value).

Do not break row elements at spaces

The goal of this commit is to make painting faster by reducing the
number of strings to paint. To this end, it is necessary to include
spaces in row elements.

Also importantly, this commit should fix existing problems with line
breaking in chinese text.

* TextMetrics::breakRow: do not do anything special for word separators.

* Row::add: when adding a character to a row element, keep the string
  width updated. If need be, it is possible to tweak this by updating
  every 10 characters, for example.

* GuiFontMetrics::breakAt (new): use QTextLayout to break text either
  at word boundary or at an arbitrary width.

* Row::Element::breakAt: use the above method.

* Row::shortenIfNeeded: simplify now that because there is no need for
  handling separator elements. This will be taken care of by the
  improved breakAt.

Two things remain to be done:

* remove all traces of separator row element

* re-implement text justification.

Support Photos in RasterImage template

Actually photos (i.e. .jpg files) where supported previously, but for pdflatex
output an unneeded conversion to png was done. The RasterImage templates
behaves now exactly as InsetGraphics for these files: If the input format is
jpg, use that for pdflatex, else convert to png.

Replace LaTeX logo with plain text

This is another advantage of the new logo insets: We see in LyX where they are
inserted for the output. In these two cases, the text describes keywords of
the external template configuration file language, so these should not be
typeset as logos.

Update sk.po

Changelog-UserGuide-LyX_22x.txt: add a todo note

for new external template

Please also announce new features here:
http://wiki.lyx.org/LyX/NewInLyX22

Add template for vector graphics

The difference to bitmap graphics is that these will be included as PDF for
pdflatex, so the vector properties are retained if a suitable conversion path
exists.

Rewrite RowPainter::paintOnlyInsets to use row elements

Store InsetGraphics bounding box in parsed form

This is like InsetExternal does it, avoids some string parsing, reduces the
amount of code and makes it more robust.

Remove relative units from bounding box

When adding units to the external inset bounding box I overlooked that
InsetGraphics supports only bb, cm, mm and in. Therefore I added too many
units for the external insets. We are not as strict as the graphics inset
and allow all absolute units, but no relative ones.

LengthCombo does not depend on VSpace

so use the right include

Devirtualize some methods

These do not need to be virtual, since no class inherits from LengthCombo.h,
and they do not exist in the base class either.

Forgot that in 81e592603

Rewrite Cursor::posVisToRowExtremity without Bidi class

It is trivial to implement it using TextMetrics::getPosNearX.

Some fixups to row margins

In breakRow set left and right margin properly for RTL paragraphs.
Remove corresponding code from ComputeRowMetrics.

In row painter, check the use of left and right margin depending on
context. The problem in the original text is that the various
leftMargin() methods actually represent right margin for RTL
paragraphs. This should be fixed eventually.

ar.po: update from Hatim

new French translation of the PDF-comment example file

Fix logic error in on-screen font shape

This made text in smallcaps+emph display as upright instead of plain
emph. There is no latex font for smallcaps + emph, but it is not a
reason for displaying it wrong.

Changelog-UserGuide-LyX_22x.txt: add a todo note

EmbeddedObjects.lyx: update all language versions after [d16a95af/lyxgit]

Please use change tracking when modifying the docs

Remove old cruft

There are many things that are not necessary anymore:

* remove methods paintText2, paintFromPos, leftMargin, paintInlineCompletion

* Remove use of Bidi class.

* add a Font and a Change parameter to paintInset.

Also remove a comment that describes what we have just done.

Rely on Row elements to draw rows.

The new version of paintText is quite short. Elements are treated one
by one. They are already in the right ordering.

The other changes are:

* a new paintChange method is added (using code lifted from old version)

* in paintLast, the support for end of paragraph markers is removed
  (this is already part of the Row object.

Code cleanup in RowPainter

* move some code that handle selection to paintInset()

* change the arguments of paintSeparator()

#9496 add a check for requested thesaurus files if thesaurusdir_path of lyxrc is empty
This path configuration variable is used for user defined location.
There are other possible places for the thesauri like mythes installers or bundled with LyX.

Support units in external inset bbox

This brings the external inset on par with the graphics insets as far as the
clipping option is concerned. The graphicxs package supports both: A bounding
box without units (which means that bp ia assumed), and a bounding box with
units, so we can simply output the values including the units.

Update default path prefix for Mac OS to include the texbin folder inside the TeX Library path and remove old powerpc path entries.
The upcoming Mac OS version 10.11 (El Capitan) will not allow TeX the installation of links inside the /usr directory anymore.

#9496 factor out getting language from LFUN args or cursor position in speller LFUNs

VCBackend.cpp: addendum to [f51d7fdd/lyxgit]

Remove InsetGraphics noUnzip parameter

Being able to compile document with zipped .eps files was a useful feature of
the graphicxs package 20 years ago, but the LyX support is no longer relevant:
- The flag is ignored if preview is on
- If pdflatex is used then uncompressing happens during the compilation anyway
- If set, the flag prevents LyX from issuing proper error messages if
something with the image is wrong
- For hard disk capacities from 20 years ago not uncompressing is a useful
feature, but for current hard disk capacities it does not matter
- The external inset does not have it, and if we want to merge both insets
one day we would need to implement it there, which is even more difficult
than in InsetGraphics

Update comment.

End all Undo::Private members names with an underscore

Move some functions out of Bidi.*

The functions reverseDirectionNeeded() and reverseDirectionNeeded() do
not rely on the Bidi class. The first one is changed into a Cursor
method, and the second one is replaced with explicit code.

Compilation fix for windows

Since commit 7ac70092, lyx::dispatch returns a DispatchResult const &,
which is ignored in the cases below. Nevertheless, the windows
compiler complains that it does not know this type. Gcc and clang can
deal with it without problem.

Remove extra row painting added by horizontal scroll branch

The bug workaround added an extra repaint, which can be very bad when
editing large tables.

It turns out that the bug this is trying to fix is due to the handling
of LFUN_LINE_END in InsetMathGrid. Adding the same code as in
InsetMathNest fixes the problem.

The workaround can therefore be removed.

Rename paintTextAndSel to paintStringAndSel

When adding completion to a row, add also the unique part

The unique and non-unique parts of the completion are show in different colors.

Avoid crash when inserting space in the middle of misspelled character

This is a fixup to commit 4796e6b3.

Add "TODO-comments" with link to textcyr/textgreek tickets.

Add links to tickets #9637 and  #9681 at places that require
changes in the code and/or comments.