installer: update the packages list

- there are some new packages required by LyX's supported document classes and example files
(this list is only used when LyX is installed the first time on a PC together with MiKTeX to shorten the installation time)

installer: new release because of bug #9733

Fix languages in math docs.

These were either unneeded or used the old german spelling.

Fix cursor movement for large logos (#9628)

Previously, if one clicked onto a large non-editable inset like the new LyX
logo inset, the cursor was always positioned in front of the inset, even if
the click was almost at the back edge. Now the cursor is positioned at the
correct edge. I tested this also with RTL contents, where from means right
and back means left, but the inset anchor position anchor point is still
at the left, and the right edge is dim.wid pixels to the right of it.

Disable assigment operator

Forgot that in cea2d71e641

Comply with rule-of-three

The rule-of-three says that if any of virtual destructor, copy constructor
or assignment operator needs to be manually implemented, then all three
should be implemented. Otherwise you can get subtle bugs which can be
difficult to find. In the changed classes, changing a copy-construction to
an assignment would have had surprising effects. Now they all behave
consistently.

Move comment where it belongs

Remove C-style casts found by cppcheck

Fix LASSERT correction

Found bx cppcheck:
(style) Assignment of function parameter has no effect outside the function.

Remove unused variables

Found by cppcheck: (style) Unused variable: x

Fix duplicated pointer check

Found by cppcheck: (style) Same expression on both sides of '&&'.

Fix copy-paste leftover

Found by cppcheck: (style) Same expression on both sides of '&&'.
I deduced the correct if-condition from the other places where theLaTeXFonts()
is called.

Add missing intializations

These were found by cppcheck:
Member variable 'x' is not initialized in the constructor.
The crash #9788 would not have happened if this had been done earlier.

Fix multicolumn regression (bug #9788)

The test case did show several problems:
- The alignment argument was not parsed correctly if it was not in braces
- There one column too much created, since I did not take into account that
  the current cell must bge replaced by the multicolumn cell
- If the last line of an array contained only an empty multicolumn cell, then
  the complete multicolumn was swallowed
- The decision whether to output the column separator & was sometimes wrong
  for multicolumns

PDF-comment.lyx: new German translation

- also disable PDF-reply since this never works correctly and it could even destroy the whole PDF and exceeds the TeX capacity
- update the french version accordingly

Initialize member

This fixes the crash of bug #9788. However, the misparsing of \multicolumn
is still there: LyX thinks that the array has three columns, it inserts an
additional one before the multicolumn.

Update LYX_DIR_21x environment variable

Fix typo

Found by gcc: operation on 'preview_ptr' may be undefined [-Wsequence-point]

Document how to use Qt 5 with cmake

Fix display of perenthesis in Hebrew documents

This was a regression from 2.1.x. I failed to copy the horrible hack that was present for the special case of () in Hebrew.

There is a real need for someone who understands RTL language stuff to fix this. Currently () are wrong in .lyx files IMO. We should not have to swap them for display.

Small code cleanup

Get rid of old Bidi code

This removes the old implementation of Cursor::getSurroundingPos, that has been superceded in commit .

As this was the last user of the Bidi class, this can be removed too.

Avoid the use of a static NSAutoreleasePool.
This seems to cause crashes on exit from time to time.

Add missing adjustment of end_pos if given length exceeds last pos

Fix "dead store" warnings of Xcode analyze tool

PDF-comment.lyx: some corrections

- also: de.po: fix a typo

Fix cppcheck portability warning

(portability) Assigning a pointer to an integer is not portable.

Revise commit 7a5eff56

Courtesy of the whitespace police.

Set minimal width of fixedwidth collapsable insets.

Intention behind this is actually Box inset being too tiny
to reasonably contain other insets like ERT.

http://www.mail-archive.com/lyx-devel@lists.lyx.org/msg188987.html

Add TOC of external materials. (#7790, partial)

Known issues (non-regressions):

* The TOC is not updated immediately after an external material inset is
  inserted. See TocBackend::updateItem().

Consistency of ellipses across the UI

Use the function support:truncateWithEllipsis() to shorten a docstring with
... at the end. Actually we use U+2026 HORIZONTAL ELLIPSIS instead of "..." when
automatically shortening strings. This is to be consistent with Qt's own
truncation and is much nicer on the screen.

This includes the bugs #9575 and #9572 regarding broken text elision in the
outliner.

Known issues (non-regressions):

* TocBackend::updateItem() should be rewritten to update all TOCs. (#8386)

* "..." should be replaced with … everywhere else on the interface (including
  translation strings).

* We should prefer to rely on QFontMetrics::elidedText() to truncate strings
  with an ellipsis whenever possible, or an equivalent for the buffer view
  dependent on the font metrics. See the warning in src/support/lstrings.h.

Remove PainterInfo argument to checkCursorScrollOffset

This will allow later to move its invokation to processUpdateFlags.

Also, remove wrong use of Update::SinglePar flag.

More updates to painting analysis

Update file from Abdel's comments and some of my own thinking

Small typos in PAINTING_ANALYSIS

Fix #9726: Preview does not have correct font size

Fix #9490: SIGSEGV involving LyX macros --- A reproducible MWE (part II)

Update sk.po

Amend d5a5fbb8e (cosmetic)

Bug #9579

Add a file that describe the current state of painting

This is a work in progress intended to start collective work towards improving the performance of our painting process.

The intent is to make it a living document that is updated as code evolves.

Small BufferView cleanup

Rename fitCursor() to needsFitCursor() to show that it does not do any
action. Constify.

Update a few comments.

Example Emacs configuration

Fix compilation with MSVC in c++98 mode

Improvements to cursor highlighting in the source panel

* TexRow now computes rows from a DocIterator. In practice, the cursor
  highlighting is now correct inside insets, it is no longer restricted to the
  topmost level. It certainly also makes forward-search more precise.

* Added the option to disable a texrow when not needed, for perf.

* Fixed a bug where the last paragraph was not properly highlighted.

Limitations:

* TexRow still does not handle: math (e.g. multi-cell), sub-captions, inset
  arguments.

Enhancements and bugfixes to the TOCs

* New TOC "math-macro". This means that math macros can now be accessed in the
  outline pane in their order of appearance or in alphabetical order, and can be
  searched using the filter.

* Lists of floats now show subfloats deeper in the navigation menu

* The arbitrary 30 element cut-off after which nothing is shown except "Open
  Navigator..." is removed. Menus now have no limit in size, so Qt may display
  them scrollable. In exchange, we always show "Open outliner..." at the
  beginning. I tested for performance issues with a rather complex document and
  it is fine; but this does not exclude corner cases with lots of TOC entries of
  a certain kind. If necessary, populating the navigation sub-menu should be
  delayed like the main menu.

* Elements that do not contribute to the output (e.g. in a note, a disabled
  branch) are now preceded with a symbol indicating this status. (The machinery
  was already there; I wonder why it was not implemented already.) I have chosen
  U+274E NEGATIVE SQUARED CROSS MARK.

* Fix the contextual menus in the outliner (bug introduced at 94e992c5).

* Toc item now move to the caption when present, but first center on the float,
  to prevent the situation where the caption is at the top of the screen and the
  contents of the float is off-screen above the caption.
  (Internally, the action of the toc items can now be customised)

* Fix the LyXHTML output. Disabled captions no longer appear in the list of
  figures.

Fix bugs #6078 and #9364

Remove unwanted clearSelection()s in MathData::updateMacros(). These calls broke
text selection with keyboard and mouse, search-and-replace, restoring selection
after Undo, etc. in a document with math macros since 1.6.0. (Regression at
6aa54673 and 12314897)

I do not know the purpose of these calls, but the selection code has been worked
on since, and I cannot produce undesired behaviour after removing
them.

Rewrite Cursor::getSurroundingPos without Bidi class

New method TextMetrics::findRowElement, excerpted from CursorX.

Reimplement getSurroundingPos using Row information. This is easy when
the cursor is inside a row element. At row element edges, different
situations can occur; hopefully all these situations are taken into
account.

Rename the old getSurroundingPos to getSurroundingPosOrig and
transform getSurroundingPos into a wrapper that compares the two
methods. This will be removed when we are confident that the new
function is equivalent to the old one.

It will then be possible to remove also the Bidi class (at last!).

Repeat after me: into account, into account, into account

PDF-comment.lyx: describe a further feature

- also some corrections

Cmake build: Remove support for concept checks.

See also commit 7f4101a

Fix some compilation problems in C++11 mode

Only enable std::regex when the header <regex> is present (fix Mac OS X issue).

Remove support for concept checks. The page
  https://gcc.gnu.org/onlinedocs/libstdc++/manual/concept_checking.html
says about concept checking:

  Please note that the checks are based on the requirements in the
  original C++ standard, many of which were relaxed in the C++11
  standard and so valid C++11 code may be incorrectly rejected by the
  concept checks. Additionally, some correct C++03 code might be
  rejected by the concept checks, for example template argument types
  may need to be complete when used in a template definition, rather
  than at the point of instantiation. There are no plans to address
  these shortcomings.

Therefore it seems reasonable to remove our support for this.

Remove support for precompiled headers for autotools

This was turned off by default for a long time, and it does not seem
to have any use these days.

Change --enable-cxx11 configure option to --disable-cxx11

Typo in tutorial.

PDF-comment.lyx: fix 2 typos

- de.po: correct a translation

UserGuide.lyx: accept Stephan's changes

- port one change to the other language versions

Correct many different typos and logical errors.

installer: update to latest ImageMagick

finally the IM people fixed the PDF image conversion problems so that I can update IM

Cmake build: Warning if system boost and stdlib-debug are used together
Change according to commit d95c187b5

Give a warning when system boost and stdlib-debug are used together

In general this would lead to an immediate runtime crash because the
runtime checks of libstdc++ change the layout of some STL objects.
Therefore, this will only work when the boost libraries have been
compiled with this same flag. At this time, it is not known whether
any linux distribution contains such libraries.

Fixes bug #9736.

Make text insets wide as soon as they contain a display inset

This also handles the case of a paragraph ended by a newline.

Fixes bug #9757.

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