ctests: invert sys font tests of colored-boxes.lyx

This example file was just committed and the system font tests do
not pass because of the user preamble code.

Add cursor<->row correspondance tracking for tables and subcaptions.

Please use the new function otexstream.append(str, texrow) to append an
odocstringstream with texrow information to the output when outputing to a
string buffer (e.g. case of subcaptions).

Improve the TexRow Cursor->Row algorithm for selections.

Selections were incorrect after the addition of math due to the invalidation of
an invariant.

Remove a deep copy of MathData in lyx::write

This deep copy used to mess with the unique identifier: what TexRow saw was
different from the original uid. There may also be performance improvements.

(Using Georg's suggestion)

TexRow info in source panel and gotoCursor() for debugging

These features are active in DEVEL_VERSION when Debug is set to LATEX.

1. The TexRow information is prepended to the source panel.

2. Clicking on any line in the source triggers reverse search. (This would be an
interesting feature to implement on the user side, but we need a proper LFUN.)

Adding TexRow information on math latex output (#4725)

WriteStream is now built from an otexstream instead of an odocstream, and
therefore counts lines in a TexRow. Calls to TexRow are added in relevant places
in math insets.

This finishes adding line tracking for math in the source panel and for forward
search.

Add math cell positions to TexRow

This is preliminary work for extending the cursor<->row tracking to math.

TexRow used to associate, to each row, a location id/pos where id determines a
paragraph and pos the position in the paragraph.

TexRow now associates to each row a list of entries, text or math. A math is a
pair uid/idx where uid will determine a math inset and idx is the number of the
cell.

The analogy id/pos<->inset/idx works better than the analogy id/pos<->idx/pos,
because what matters for the TexRow algorithm(TM) is the behaviour in terms of
line breaks.

This only improves the source view and the forward search, not the error report
and the reverse search (though this could be easily added now).

Add a unique id to math insets

The id is just the memory address.

The status bar now spits out the math inset uid information when in a math cell
in DEVEL_VERSION, like it already does when in a paragraph.

This is preliminary work for extending the cursor<->row tracking to math.

Splitting otexstream into otexrowstream and otexstream.

otexstream used to count lines to build a TexRow, and some other things. The new
class otexrowstream has the line counting feature of the previous otexstream
without other stuff. otexstream is now a subclass of otexrowstream that has the
same features as before.

This is preliminary work for extending the cursor<->row tracking to math.

ctests: invert a test that never passed

The H-P-statements example was recently introduced (a610e29c) and
the XeTeX test with system fonts has never passed.

ctests: invert a test that never passed

The ICUr template was recently introduced (5189d7be) and the XeTeX
test with system fonts has never passed.

tcolorbox.module: uniform whitespace

colored-boxes.lyx: new example file for the tcolorbox.module

- also improve the tcolorbox.module

This is a fileformat change - only to be able to revert the new styles defined in the module.

Fix language end command

The code that sets open_polyglossia_lang_ is not only executed for polyglossia,
but also for babel, so we have to use the correct language end command.

UserGuide.lyx: port changes from branch

EmbeddedObjects.lyx: revert unintended part of previous commit are commit correct stuff

- sorry, after some moths without LyXing I forgot most of the complicated git branch handling stuff

EmbeddedObjects.lyx: update appendix A

Since a while now we can translate the unit descriptions. For some special applications it is also necessary that the users know the LaTeX command of the relative units.

- also fix a typo in Customization.lyx

sk.po: Use '…' instead of '...' where appropriate

Remove unneeded whitespace

Remove confusing parentheses

Fix regression of 359aef92f87

The context menu did no longer work for some insets, since it requires the
cursor to be in front, and editXY() is also used to determine the inset for
the context menu. Now the cursor is corrected if needed.

chkconfig.ltx: add packages supported by LyX

- also update Win installer package list (trimspaces is required by tcolorbox but not listed as required in the tcolorbox manual)

RELEASE-NOTES: LyX should work with Qt >= 4.5

Fix std::string docstream output error

This was made visible by aab1b145a57c, since xhtml export for
lib/doc/Additional.lyx caused an exception. However, the cause for this was
already present earlier: All attempts to output a std::string to an
odocstream resulted in trying to change the encoding of the stream instead,
since there is no operator<<(odocstream &, std::string) defined, and an
implicit conversion to SetEnc happened instead.
This is fixed by making the SetEnc constructor explicit and adjusting all
code parts that did not compile anymore after that. The parts of the code
that did use the wrong output operator were the std::string version of
htmlize() from output_xhtml.cpp and all changed parts in the other .cpp files.
I also removed the std::string versions of html::htmlize() and
html::cleanAttr(), since it was difficult to see which encodings were used
with these. Now we are always explcit when using html::cleanAttr() and
html::htmlize().

Update RELEASE-NOTES for Qt advice and policy

Remove variable y_top that was computed but not used.

Correct early return position for if use_pixmap_cache_ check

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