Look for svgz images as well

Almost all toolbar images are now in svgz format, bu thos were not found by
the scripts.

configure: Add export format for 2.2.x

Adjust generate_symbols* to db363ab1

The image map was renamed, so it was not found anymore.

Make unicodesymbols.py python3 compatible

Update gen_lfuns.py to current format

And run it to update LFUNs.lyx. This overtakes the  syntax fix from b356df7fc
as well.

Make gen_lfuns.py python3 compatible

Also read files with universal line endings and write files using native line
endings, so the script can also be used on windows without changing the line
endings.

Remove convert_kmap.py

This was used to convert LaTeX symbols in kmap files in 2007, we do not
need it anymore.

Get rid of pseudo diffs when remerging strings

cmake sorts the input files for lyx_pot.py internally, but autotools use a
shell pattern like *.ui on the command line, so the order may be different
on different machines. It is more robust not to require any sorting from the
caller, so lyx_pot.py sorts now internally.

update test references for ba2b86fa

Work around limited SVG support (bug #9778)

Qt only supports SVG 1.2 tiny, so we prefer to convert on our own if an
explicit converter is defined and the converter cache is used (otherwise
the conversion would be too expensive).

Update sk.po

typo

Take inset dimension into account in row signature

In some cases, the insets may change height or width without changing
the other apsects of the row.

Fixes bug #6991 and #10182.

Comment only

Introduce basic support for microtype.sty.

https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg192743.html

Support a changed log message in newest LuaTeX

The version of LuaTeX that ships with TeX Live 2016 now gives the
following message after processing a document that yields no pages
of output:

"warning  (pdf backend): no pages of output."

The lowercase "n" in "no" is a change that caused our parser not to
pick up the message.

Output file from (xe|lua)latex is PDF, not DVI

A few parts of our code depend on correctly identifying the output
format of LaTeX commands. One specific bug is that because the
output file was not correctly set, it was not removed after an
error. For example, this commit fixes the following bug:

1. Create a new document that contains "hello\blah" where \blah is
in an ERT box.
2. Compile with PDF (LuaTeX). You'll get an error because of \blah.
3. Close the error dialog.
4. Remove the text "hello" and compile again with PDF (LuaTeX).

The error dialog is shown and the "Show Output Anyway" button is
enabled. If you click it, it shows the previously compiled PDF (with
the text "hello"). With this commit, the button is correctly
disabled (and the output file is deleted).

Generalize a comment

An output file does not have to be a DVI.

Win installer: port last change

Tutorial.lyx: port last change

Export svg in xhtml, not svgz (bug 10160)

LyX did not distinguish compressed and uncompressed svg files previously.
Therefore XHTML export of vector graphics did use svgz images directly, which
is not supported by browsers. If svg and svgz are treated as two formats then
all works fine. This is also consistent with the loadable image formats
reported by qt: It reports both svg and svgz.
The gunzip dependency in converters is not new (it is already used internally),
but the gzip dependency is new, so it might not be available on windows.
This is not important at the moment, since we do not yet need to convert svg
to svgz, I only added the converter for completeness.

Do not duplicate jpg format

Some qt versions report both "jpeg" and "jpg" as loadable file extensions.
In this case the jpg format was added twice previously. This does not happen
anymore with the new code, and it works as well if only "jpg" or only "jpeg"
is reported.

Improve debug output

Now we have only one line per format, not two. The output looks the same now
as in GuiApplication.cpp for the qt formats.

Win installer: security issues in ImageMagick require new installer

Update imported gettext code from gettext 0.19.7

This might help with the spurious .pot file update error in 'make distcheck'.
At least it does not break anything.

glitch in lfundoc

de.po: some translations

ErrorListUi.ui: correct an expression

Force native line endings for .po files

As discussed on the list some time ago. cmake produces .po files already in
native line endings. Only autotools on mingw might produce wrong line endings
now, but I am pretty sure that nobody updates .po files using autotools on mingw.

Allow automatic text direction in Painter::text()

It is wrong to assume that direction is left-to-right when no indication exist.

Add a new enum with values LtR, RtL and Auto to be used as argument of
the private text() methods. When direction is Auto, let Qt decide how
the string shall be layed out.

Fixes bug #10169.

Update sk.po

Cmake export tests: Added testcases for erroneous xhtml exports

Win installer: 2 bugfixes

TocWidget: fix an erroneous collapse and optimise updates based on profiling

TocModels::reset() in GuiView::structureChanged() collapses the TocWidget, and
therefore requires an update right after, which was missing.

In fact, profiling TocWidget::updateView() shows that delaying the update is
good only for fast keypresses (essentially movement). It costs 5% of a
char-forward operation in a document with approx. 100 table of contents
items. The update optimisation has been rewritten to take this data into
account.

TocWidget: fix part of #9825

TocModel: clean-up

GuiDelimiters: display at least (and at most?) 4 columns

GuiDelimiter: Synchronise the vertical scrollbars

ShortcutPrefs: hide empty lfuns for which there is already a binding

ShortcutPrefs: clean-up

Show names of debug modes in the debug panel

Sometimes it's hard to know which is which from a translation of the description
alone.

UserGuide.lyx: update 2 images

Update PAINTING_ANALYSIS

Remove all the tasks that have been performed.

Split TextMetrics::setRowHeight in three parts

The new functions parBottomSpaging and parTopSapcing return below/above each paragraph.

This allows to remove the TopBottomSpace argument and makes the code a
bit clearer.

Do not require an extra pit parameter when a row is available

Now that Row has a pit() member, it is possible to use it instead of passing an extra pit_type parameter to a function which already has access to a Row.

RowPainter const cleanup

Change the various paint* helpers to take a single row element as argument.

Do not update x_ in the various paint* helpers. Constify them. Update x_ in paintText and paintOnlyInsets instead.

Remove an empty call to paintForeignMark in paintInset (the call did nothing since orig_x == x_ at this point).

Update PAINTING_ANALYSIS and add a new task

Set vertical margins in redoParagraph, not setRowHeight

It is actually easier to set the 20 pixels margin in redoParagraph, since it is not necessary to take newlines in account when deciding what is the real last row of the paragraph.

Moreover this solves the following bug (present in 2.1.x too): when the document ends with a newling, the bottom margin disappears.

Update PAINTING_ANALYSIS with new tasks.

Cleanup Painter text() API

* remove optional arguments to the helpers that use a FontInfo
* add a textwidth argument to the text() methods that are used by rowpainter.

Now textwidth is only computed if a null value was passed to the
text() method. This means that in the use case of rowpainter, no
textwidth needs to be computed.

Make the different Painter::text void methods

The textwidth value that was returned is not used anymore. Now in some
cases we can avoid to compute it at all.

Do not use the return value of Painter::text for logos

This will allow to get rid of this return value (and avoid computing it).

Rewrite setRowHeight using row information

The initial values for maxasc and maxdes (renamed from maxdesc) is obtained as a maximum of max ascents/descents of all row elements.

This allows to get rid of Paragraph::highestFontInRange and FontList::highestInRange.

Some auxilliary variables declarations are also moved to where they are needed.

Simplify redoParagraph by merging duplicate code

Let breakRow return a boolean indicating whether an additional row is
required (after a newline) and use that to replace the code that added
an extra row when a paragraph ends with a newline.

Update PAINTING_ANALYSIS

Some things were wrong, and some new ideas are added (some are done in
this branch already).

Make the non-drawing cases faster in TextMetrics::drawParagraph

There are two main cases:
* when drawing is disabled from the start, use a simplified code that only paints insets (in order to cache positions).
* when the row is not visible, do the same.

The goal of this optimization is to be able to always run a no-drawing draw after the metrics have been computed.

Move some horizontal scrolling code from TextMetrics to BufferView

It is better to have all the code in the same place, and it will avoid code duplication later.

Only add inset postion to cache in paintInset

It was also added in paintText and paintOnlyInsets.

Cmake xhtml export tests: Missing script for commit 416242476289d423eb392345555f3dad941df544

Cmake xhtml export tests: Use command xmllint to test the export result

Dont use invalid tag '<pre></pre>' in xhtml export

Cmake tests: Allow test of LongestLabelWithUnderscore_xhtml

Cmake: Allow test for X11 with QT5.6

Used for key-test tests.

UserGuide.lyx: update 2 images

CMake: add 2 build scripts to compile LyX with Qt 5 and MSVC 2015

tufte-handout.lyx: add the lost \origon tag

That the origin tag is automatically removed sucks. It consumes a lot of time to re-add it again and again.

tufte-handout.lyx: remove some unnecessary TeX code

Merge branch 'master' of git.lyx.org:lyx

Win installer: push the changes that are already part of the 2.2.0 installer

Correct wrong path for included lyx-file

Although the fix affects only a content of a LyX note, it is
still surprising to get a dialog-window demanding creation
a new document if trying to edit the subdocument.

Fixup 1b2aad9 : Cleanup of spacing in mathed

Now that 5mu are added on each side of : (because it is a relation), there is no need to do the same in the definition of \vcentcolon.

This is closer to what is done in mathtools.sty.

Cleanup of spacing in mathed

This is a first cleanup step. More complex rules have to be
implemented on top of this.

Use proper spacing \thinmuskip, \medmuskip and \thickmuskip instead of
ad-hoc values.

Rename isRelOp to isMathRel and introduce isMathBin and isMathPunct
(for InsetMathChar and InsetMathSymbol). Update the categories of
characters in InsetMathChar according to LaTeX source (fontmath.ltx).

Set correctly the spacing around mathrel, mathbin and mathpunct
elements. Use \thinmuskip around MathDelim instead of a hardcoded 4.

This is related to bug #8883.

Improve documentation of LyX' "smart quotes" feature.

Remove extra spacing around InsetCommand buttons

There is already a spacing of 2 pixels on each side of a button (e.g. collapsed inset). There is no need to add one extra pixel for command insets.

Fixes part of bug #10149.

Fix drawing of buttonText (enforce same left/right spacing)

The computation of the width of the button was wrong. If <--> stands for TEXT_TO_INSET_OFFSET/2 spacing, and if `[]' marks the button's limits, then the intent is
  <-->[<-->button text<-->]<-->

Therefore the physical grey rectangle width is
   width - Inset::TEXT_TO_INSET_OFFSET

With this change, the spacing on the right of the button is not larger than the  left one.

Fixes bug #10147.

Cleanup handling of LFUN_LAYOUT_PARAGRAPH in getStatus

The way it works is:
* the inset defines allowParagraphCustomization() correctly
* Text::getStatus acts on it.

Note that, in Text::getStatus, testing for cur.inset().allowParagraphCustomization() does not make much sense, since one should pass the cursor idx as parameter. Actually, for some reason the safest bet is to use the owner of the Text object as inset.

Cleanup handling of LFUN_LAYOUT in getStatus

The way it works is:
* the inset defines forcePlainLayout() correctly
* Text::getStatus acts on it.

Note that, in Text::getStatus, testing for cur.inset().forcePlainLayout() does not make much sense, since one should pass the cursor idx as parameter.

There are many other lfuns that do not have to be handled directly by insets. InsetScript in particular has tests for way too many lfuns.

Rename badly named variable

Remove variables set but not used

This was found by cppcheck.

Move one Text::setCursor instance to CursorSlice

This method did access more CursorSlice than Text. It is only a setter for
CursorSlice with some bound checking. The new signature is
setPitPos(pit_type, pos_type).

Variables of type pit_type should be named pit, not par

It is easier to use always the same conventions for naming.

Rename Cursor::setSelection(bool) to selection(bool)

The old name would be confusing wrt setSelection(), which does additional checks.
This one is a pure acessor, and the more complete methods are
* setSelection(), which avoids empty selections
* clearSelection(), which resets anchor, and sets word selection and mark more to false.

Most of the code should use these two instead of selection(bool), but this is for later.

Avoid incorrect "Autocorrect Off" message

FindAndReplace: reorder includes

TextMetrics: Use shorter names for font metrics

This keeps code easier to read.

Cleanup bruteFind.

Remove unused bruteFind* functions

And rename bruteFind2 to bruteFind.

Remove unused CursorData::logicalpos_

This was introduced in 41ecabf5, probably by mistake.

Directly pass a Row::Element to RowPainter::paintInset

Remove unused TextMetrics::maxWidth()

Use isMainText() instead of doing explicit tests

We do not use <boost/scoped_array.hpp> anymore

The code using it  was removed at 81959251.

Remove support/metahash.h

It turns out that it did not take off since introduced in 2011. It is better to remove it and the associated boost headers (extract.sh was run against boost 1.60 to do the update).

Since we will move away from several boost classes when transitioning to C++11, it is good to start by removing lesser used ones.

Do not use of boost::tokenizer

It is easier to use instead getVectorFromString for the use we have of this tokenizer. The two places are environment.cpp (path stuff) and qt_helpers (file fileters). The new code is much shorter.

This allow to remove boost/tokenizer.hpp and friends from our boost tree.

Remove boost/format support

Only ChkTex.cpp used it, whereas the rest of our code uses the simpler bformat.

Remove declaration of classes that do not exist

Some headers contain
  class Foo;
whereas there is no class Foo.

The list of class statements is given by
  classes=`git grep  '^\(class\|struct\) [a-zA-Z_:]*;' src | sed 's/^.* \(.*\);/\1/'|sort -u`

The ones that are useless are:
  for c in $classes ; do grep -r "\\<$c\\>" src| grep -vq '^[^:]*:\(class\|struct\) [a-zA-Z_:]*;' || echo "$c"; done

\\pagebreak is a fragile command in general

As such, it shall be protected in moving arguments, such as sections.

Fixes bug #10092.

Fix getStatus for math-mutate and math-display

Those two functions used two different hackish and buggy
implementation to know when the function is disabled. Replace that by
asking the containing inset whether it accepts inserting display math
inset.

Fixes bug #10033.

Do not use 'u' flag for ar

The Reproducible Builds effort (https://wiki.debian.org/ReproducibleBuilds) in Debian (at least) means that 'ar' is built in deterministic mode as default: all timestamps are set to 0.

This is not compatible with the use of the 'u' flag, and therefore ARFLAGS has to be changed from 'cru' to 'cr'.

This gets rid of the harmless but annoying warning
  ar: `u' modifier ignored since `D' is the default (see `U')

Remove unused include file.

Correct path names were to look for RPM based dictionaries for hunspell on Linux.