New helper method Row::Element::isRTL()

de.po

Add assertions, just to be sure.

Fixes coverity issue #111941.

Win installer: preparations for a new build

- the long awaited MiKTeX installer is available, this fixes many installation errors in LyX is installed for the first time (I will provide a new installer build as soon as I find more time)

Remove unused variables found by cppcheck

There are still a few warnings of the kind
(style) Variable 'x' is assigned a value that is never used.
since I did not touch code where I was not sure whether there might be a real
bug, and I kept some for symmetry reasons as well.

Fix missing xhtml output found by cppcheck

Actually I don't know if this operator is used at all, but if it exists then
it must output something.

Fix wrong label found by cppcheck

It was not intended to ignore the return value of expandParagraphLabel().

Forbid wrong compiler generated copy constructors

This fixes cppcheck warnings (style) 'class x' does not have a copy constructor
which is recommended since the class contains a pointer to allocated memory.

Use Require instead of AddToPreamble

This plays more nicely with other modules requiring enumitem as well.

Amend 94e992c5

Fix compilation with --disable-cxx11

Add comment regarding fall through.

Coverity issue #111940.

Update sk.po

Create module for paralist.sty

paralist.sty extends the standard list environments by some more compact
versions. Support for this has already been requested 15 years ago, and
now I needed it myself.

Work around encoding bug

It looks like the author names are sometimes not encoded correctly.
We'll need to find out where this happens, but for now just fix the symptom.

Fix cppcheck variable scope warnings

These were all flagged by "(style) The scope of the variable 'x' can be reduced."
Narowing the scope improves readability, and if it is in a loop then the
compiler will be clever enough to produce efficient code, we do not need
manual optimization for POD types.

Refuse to run tex2lyx tests with in-source builds

As Günter found out running the tex2lyx tests overwrites the test references
if the build directory is identical to the source directory. Therefore the
tests would always pass, but git diff would show a non-empty diff if the tests
should have failed. Since it is better anyway to build in a separate directory
we simply do not support srcdir = builddir for the tests and abort with an
error.

restore crucial float information in note insets (bug #9760)

New class InsetCaptionable

This generalises the new use of TocBuilder (97e8101) to listings and wraps and
removes some duplicate code.

Make sure that we will never write an empty float type again in the future
(#9760)

Better construction of the TOC for floats and captions

We introduce TocBuilder for building TOCs that take into account both float
insets and their captions.

* Floats without caption are shown with their content.

* Floats with a caption are shown with their caption, but clicking the entry now
  correctly moves to the float and not to the caption.

* Subsequent captions produce additional entries in the TOC.

* Figures and subfigures are correctly ordered in the outliner.

* New TOC "senseless" for captions appearing alone (a bit like broken references
are still displayed in the menu and outliner).

* Disable LFUN_CAPTION_INSERT if there is already a caption in a listing

Known issues:

* Inconsistent output for includes located inside floats

* We should record the end of the float in addition of the beginning for a more
  accurate cursor -> outliner entry conversion

Do not show subfigures in the menu (bug #9759)

* More consistent behavour (no 30 elements limit)
* Subfloats are given depth 1
* Only show depth 0
* Always show "Open outliner..."

Support for (std|boost)::make_shared

Correct commit 92b0886: true is not True in python.

Math commands used in text mode now always use the current selection as contents
(#9762)

* fixes a bug where this was already the expected behaviour of
  math-subscript and math-superscript but failed.

* corrects the behaviour where if there is \newcommand in the
  selection, then a corresponding macro template is introduced
  instead of a math inset.

  * fixes a bug where math-display, math-subscript and math-supscript
    would also introduce such a macro template in a way unrelated to
    their function. Now it only happens with math-mode without
    arguments.

  * fixes a bug where a text that does not denote a macro definition,
    e.g. "aaa\newcommandaaa", would produce \invalidmacro.

Add some missing member variable initializations

These were all found by cppcheck. Even in constructors that are there "only
because of std containers" the class should be initialized correctly. You can
never know whether such an object does not get used, and then a nice crash
caused by dereferencing a NULL-pointer is better than undefined behaviour.

Return reference to this in operator=()

This is the recommendede signature of assignment operators and was found by
cppcheck.

Remove dead code

Thanks to Jean-Marc and coverity for finding it.

Work around a gcc5 bug

The old code produced crashes with gcc5 caused by calling the copy
constructor (see https://bugzilla.redhat.com/show_bug.cgi?id=1260976).
This has been filed as gcc bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67557
It is currently being marked as invalid (they claim the code is not valid and
produces undefined behaviour), but I don't think that this is correct.
Fortunately i does not matter for us whether the old code was valid or not,
since the new version is easier to understand and works with all compilers.

Make destructor virtual

Base classes with virtual methods should have a virtual destructor.
See Scott Meyers, Effective C++ item 7: Declare destructors virtual in
polymorphic base classes.

Fix bug #1996

\overrightarrow and related macros have a standard behaviour with respect to
sub/superscripts.

Improvements to the display of the source.

Fix bugs #6501 and #7359.

* Selection highlighting has been broken since a conflict with
  reverse-search was repaired (at 00a62b7c), is now fixed.

* The selection highlighting was not accurate, for LaTeX formats (in
  full source view), and meaningless, for non LaTeX formats.

* fix regression at cc00b9aa: force_getcontent_ was always true
  therefore the code to detect changes in the generated source was
  dead. The consequence is that the source view would jump to the
  beginning at each updateView() even if no change occurred. Cc00b9aa
  was meant to fix #5600, which I cannot reproduce with the new
  implementation.

* Various improvements:
  *  When the position-to-line conversion is unavailable (LyXHTML, LyX
     source, etc.) we focus on the first difference instead.
  *  Get some space around the cursor
  *  Respect the scrollbars
  *  Highlight with QTextEdit::ExtraSelections instead of cursor
     selection (the latter used to break syntax highlighting of the
     TeX code... which was not so much of an issue before because the
     wrong part was selected)

Known issues:
  * The highlighting is off by one line in very last paragraph of a
    document. This appears to be a bug in TexRow.cpp.
  * The highlighting is off for any kind of inset. This could be
    solved by adapting TexRow so that it accepts CursorSlices as
    and argument for the conversion to line number. (this is bug
    #4725)

Better session management for the source panel.

We now remember the previous GUI values across sessions.

Improve the performance of the source panel by using a QTimer.

Fix bug #9493. The source panel was very slow with auto-update on. Now
we use a timer that ensures that the source is generated only at
rest. The delay is short or long depending on whether we show the
source of a paragraph or the whole source.

Fix bug #9754

Don't delay the update of the TOC panel on switching documents

In fileLock, take in account the case where the file cannot be opened

overity issue 23352

iInitialize a few variables in TextClass constructor

Also use proper initializers and reorder members properly.

Coverity issue 23385

Initialize properly two members of InsetTabular

Also rename first_visible_cell to first_visible_cell_. I wonder though whether this variable is still useful.

Coverity issue 23383

Initialize MathData members

Coverity issues 23381 and 23436

Initialize pointer members

Coverity issue 23387

Add missing break in switch

Coverity issue 102058

Annotate missing breaks in switch for coverity

Coverity issues 102057 and 102060.

Annotate missing break in switch for coverity

Coverity issue 23362

Annotate missing breaks in switch for coverity

Coverity issues 23341, 23359, 32064 and 102056

Annotate missing breaks in switch for coverity

Coverity issues 23344, 23354, 23371 and 102059

Add missing break statement

Coverity issue 23338

Add missing breaks

Coveity issues 23333, 23334 and 23348.

Remove useless test

Coverity issue 23300.

Restore lyxerr number format to decimal

Coverity issue 23427

Do not pass objects by value

Coverity issue 23372.

Make the while loop actually loop

Coverity issue 23513.

Fix logic of method

The last return statement was unreachable

Coverity issue 102063.

Mark dead code as dead

Coverity issue 23511.

Add missing braces

Coverity issue 23506.

image_ready is false at this point

Coverity issue 23299.

Remove dead code

first is always false at this point.

Coverity issue 23298.

Fix unreachable code

c cannot be \\ at this point.

Coverity issue 23296.

Remove unreachable code

Fixes coverity issue 23295.

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