A first batch of potential bugs spotted by llvm/clang

There are more warnings left, but these are the easiest to fix. And some of them are real bugs (assignments with == instead of =).

Remove some header leftovers based on Scott's script hints.

False positive rate of hints is quite high. Although the includes can be
technically removed (due to other includes) they logically belong to the
header.

Script for finding unneeded headers

The script does the following:

  All .cpp and .h files in the current directory and subdirectories
  are checked to see which include statements could be omitted without
  causing a build error.

Many of these omissions would not be desired. For example, currently
if you don't include Undo.h in Undo.cpp, there is no error because
Undo.h is included in Cursor.h which is included in Undo.cpp. But
clearly we do want to include Undo.h in Undo.cpp.

See #6305.

installer: fix a typo

Don't ask for creating branches on copying

As discussed on the list. This gets rid of the unwanted message boxes when
copying the complete Userguide.

Small changes to Poor man's profiler

* report results in milliseconds instead of microseconds
* report total time spent in block, additionally to mean time
* cause compilation error with --enable-stdlib-debug

Fix bug #8627: Command line switches to change branch do not appear to work

Move the handling of branch-(de)activate(master) to Buffer. This code was moved to Bufferview in [3a03e71c/lyxgit] because a cursor was necessary to call Undo::recordUndoFullDocument(). However, it turns out that the undo code is already prepared to handle an empty cursor (and do nothing in this case).

Therefore we do that and move the branch code to Buffer where it belongs.

Note that there was a bug in the previous code that broke undo support: recordUndo should always be called _before_ doing any change.

Added regression test for simple use-case documented in Advanced F&R manual.

findadv-re-05 with ignore format fails, whilst findadv-re-06 not ignoring format succeeds, as of now.

UserGuide.lyx: fix an issue reported on the docs list

UserGuide.lyx: next revision step (chap. C) for master

Fix assertion for InsetInfos of icon types

InsetInfo inserted insets in its text without specifying a font. This
caused the fonlist to be incomplete and firing the assertion.

Fix FontIterator to not access pos >= par.size()

An iterator is typicall incremented passed the valid data into the 'end'
state. However, if this means that other functions are called with invalid
parameter, we should fix this.

This explains why fontSpan was called with pos==size() while this was not
expected to happen.

This patch reverts partly f5ad0c128aec (Jean-Marc Lasgouttes; Get rid of
annoying warning message).

Make sure MarkAsExporting is deleted before buffer

This fixes a crash when cutting something.

Make staticbuffer static again

Otherwise the number of internal buffers in incremented on each cut
action.

Clean up comments a bit.

installer: updated for dictionaries

- support thesaurus for Icelandic and Ukrainian
- update for other dictionaries

UserGuide.lyx: port recent changes from branch

Another assertion that got changed wrongly.

I've read back through that patch, and everything else looks good.

Remove all messages to user related to assertions. We'll just use
a basic message which is defined in lassert.cpp now.

Use doctring as LBUFFER() etc parameters

Strings as parameters to LASSERT should not be translatable.
But empty strings are even worse.

Another assertion that got transformed in 1b1f8dd2

Get rid of annoying warning message

French UserGuide.lyx: translations by Jean-Pierre

po: Modularisation of the diff_po.pl script

Fix assertion related to cursor anchor.

Remove assertion accidentally added during the audit.

Audit all the LASSERT calls, and try to do something sensible at
each failure.

There are several places I was not sure what to do. These are marked
by comments beginning "LASSERT:" so they can be found easily. At the
moment, they are at:

Author.cpp:105: // LASSERT: What should we do here?
Author.cpp:121: // LASSERT: What should we do here?
Buffer.cpp:4525:        // LASSERT: Is it safe to continue here, or should we just return?
Cursor.cpp:345:         // LASSERT: Is it safe to continue here, or should we return?
Cursor.cpp:403:         // LASSERT: Is it safe to continue here, or should we return?
Cursor.cpp:1143:                // LASSERT: There have been several bugs around this code, that seem
CursorSlice.cpp:83:     // LASSERT: This should only ever be called from an InsetMath.
CursorSlice.cpp:92:     // LASSERT: This should only ever be called from an InsetMath.
LayoutFile.cpp:303:                     // LASSERT: Why would this fail?
Text.cpp:995:           // LASSERT: Is it safe to continue here?

Add additional LASSERT-type macros for dealing with various kinds
of situations.

Add a bunch of comments about the different exceptions.

Simplify code a bit: If we only accept CITE_CODE, there is no need
to pass it.

Default font for an InsetLayout should be to inherit everything.
Fixes bug #8610.

Just a bit of renaming.

Fix the size of the hAlign combobox in GuiTabular

The default sizeAdjustPolicy is AdjustToContentsOnFirstShow. As the
combobox is initially filled with all items, it will not change size if we
later remove the 'At Decimal Separator' item.

CPack: Do not use CPACK_SET_DESTDIR on Windows

Fix CMake on Windows

Fix bug #3199: change-next may leave empty lines behind

Using Cursor::setCursor or even BufferView::setCursor is often a bad
idea since it does not run DEPM. In this case (and other cases in
f&replace code) it is better to use BufferView::mouseSetCursor (which
should maybe be renamed...).

converter: change 'ps2pdf13' to 'ps2pdf'

ps2pdf by default produces the PDF 1.4 format. The PDF 1.3 format was
released in 2000. PDF 1.4 was released in 2001.  LyX specified 1.3 as
the output version in 2002 (c1541c22), perhaps because at the time
PDF 1.4 was only a year old so some viewers did not support it.

Fix bug #8633: change tracking: paragraph break is restored by undo

A recordUndo call was missing in this case.

Adjust CMake for Qt5

Extend and use qt4_wrap_ui

Fix ChkTeX error reporting (#8639)

ChkTeX errors from a previous run are now cleared from the
error list if ChkTeX exits with no error.

GuiRef: only enable "Group" CB if non-empty refs

GuiRef: do not insert a category as a ref

Before, if a user had "Group" checked in GuiRef and double-clicked
on a category, that category would be inserted as a (broken)
reference. Now, when a category is double-clicked, nothing happens.

GuiRef: do not hide labels that start with ':'

If a label starts with ':' and 'Group' is checked,
the label is now shown under "<No prefix>" instead of
not shown.

Note that labels starting with ':' can be valid in LaTeX.

GuiRef: only show "<No prefix>" if non-empty

If "Group" is checked, "<No prefix>" is shown only if there exists a
label with no prefix. Before, it was shown even when it was empty and
even when there were no labels at all.

UserGuide.lyx: fix an expression as discussed in the list

port Spanish doc updates by Ignacio

Spanish UserGuide.lyx: updates by Ignacio

- also some changes to the other languages proposed by Ignacio

German Tutorial.lyx: fix a typo spotted by a user

Fix bug #8636

This was a simple logic error that crept in during refactoring:
If the format does not match, and the converted file has been read,
the string must not be read.

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

French UserGuide.lyx: translations by Jean-Pierre

Fic crash for repeated pasting from complex docs

Using a cloned buffer was an idea by Richard, and it works well.

Use MathML on the clipboard for formulas in HTML

This speeds up the copying, and although not many applications understand
MathML currently (MS word is supposed to understand it), their number should
go up in the future.

UserGuide.lyx: next revision step (sec. 3.5 - 3.7 and 6.18)

PrefInputUi.ui: change 2 strings

because all over the docs we refer to the keyboard maps as primary and secondary

UserGuide.lyx: next revision step

Fix bug found by Scott concerning copying XHTML to clipboard. We
needed to updateMacros() before doing the export, which required a
bit of code massaging.

Revert "Fix the zombie toolbar problem (bug #8520)"

This reverts commit d00ec4dececeae51c3edb032cf7e97d7bb81f5cf.

Revert "avoid redundant toolbar initialization"

This reverts commit 0f26aabeee3663f2cfaa89c983b668486610e564.

It turned out this breaks session.

fix some more typos spotted by Yuri

Update boost to 1.53.0

Whitespace

Update boost to 1.53.0

Pass the binary dir to the configure script to find tex2lyx

When using CMake, the binary files are stored in <build-dir>/bin. LyX can't fin tex2lyx with the current code. So, we have to point configure.py to explicitly look in the binary dir.

uk.po: update from Yuri for master

also fix some typos spotted by Yuri

UserGuide.lyx: next revision step

UserGuide.lyx: first step of revision

- first step of the revision of the English UserGuide from John Hudson

jasatex.layout: fix 'Acknowledgments' style

'Acknowledgments' does not belong in the title but did not override
'InTitle 1' inherited from 'Abstract'. Without this patch, the
following error is given:

  output_latex.cpp (1111): Error in latexParagraphs: You should not
  mix title layouts with normal ones.

Thanks to Hashini Senaratne for narrowing down the problem.

Implement paste from LaTeX and HTML (bug #3096)

As discussed on the list. No automatic contents detection is done, the user
needs to use the special paste menu instead. I used the new TempFile class
for safe temporary file handling.
The documentation would go into section 2.2 of UserGuide.lyx, but I am not
allowed to edit that document.

Add class for threadsafe temp file handling

FileName::tempName() is not thread safe, since the QTemporaryFile object is
immediately deleted after creating it. Therefore, another thread could create
the same temporary file in the time span before the user of FileName::tempName()
recreates it. This is not as theoretical as it may look: I observed that
repeated creation and deletion of QTemporaryFile objects always use the same
name.
This problem is solved by the new class TempFile which should be used like
QTemporaryFile itself.

Fix bug #8631: Python detection unfinished

sgml: ignore paragraphs named *dummy*

dummy is the name given in the latex backend to simple (usually single) paragraphs and since dummy is not a docbook element it is acceptable to ignore it. It is a reasonal compromise.

Make parameter more readable.

Add a DocBook -> PS converter

Before, the converter chain for DocBook -> PDF (ps2pdf) was:

  docbook -> DVI
  DVI -> Postscript
  Postscript -> PDF (ps2pdf)

sgmltools has a backend for PostScript so the first two
steps in the above converter chain are now condensed into
one by adding the following converter for docbook -> Postscript:

  sgmltools -b ps $$i

installer: fix a an UI issue

- also update to Python 2.7.4

installer: support Canadien

- support spell-checking French (Canada), see bug #8629
- prepare for LyX 2.0.6

Improve LaTeX format detection

libmime is a bit lacking here.

Also put HTML on the clipboard when copying

The HTML export is now mature enough so that it can be used to transfer
formatted text to the clipboard. This enhances interoperability e.g. with
office applications.

Work around gnuhtml2latex encoding bug

gnuhtml2latex does not handle encodings at all. Therefore the result is not
imported correctly by tex2lyx if the HTML file is encoded in anything else
than ascii or latin1 (the default of tex2lyx). The simple wrapper script
loads inputenc if needed. It may not be possible to compile the result with
LaTeX, (e.gif utf8 is used), but for running tex2lyx it will work just fine.

de and es tufte examples: change default to pdf2

Consistent with 1b7e9178

hu/Tutorial.lyx: change default output to pdf2

Remove useless test and add comment.

Do less indenting.

Remove obsolete comment.

Const.

Replacing git status --porcelain along with output parsing, with a simple git ls-files.

git ls-files tells me straight whether or not a file-name is in the repo.
I cannot say the same of git status --porcelain.
See also discussion at: http://www.mail-archive.com/lyx-devel@lists.lyx.org/msg177840.html

Avoid check mistakenly current folder for .git/.svn/.CVS, when going up the path checking for parents.

Remove partial support for non-buffer files in VCS

This reverts a small part of 0526eb9d and gets rid of the double calls of
SVN::findFile() and GIT::findFile(). Please note that git detection does
currently not work because of 26dd4d0c.

languages: whitespace only

languages: use a usable encoding for Kazakh

- we found out that Kazakh is currently not supported by babel nor polyglossia so that using Kazakh as language will break the compilation, Günter proposed therefore this encoding change until Kazakh is supported.

Hungarian Tutorial.lyx: revert last change

- this is not the way it goes, all Tutorials should stay identical as possible, moreover there is no need to disable PDF bookmarks because the document compiles without problems, moreover I still manage the documentation and as long as there is no beta for LyX 2.1 please keep the fileformat of LyX 2.0
- if you have problems compiling a file, report this as bug or the docmentation mailing list! or at least write me a mail

JSS-article.lyx: set default output to pdf2

The reasons are specified in the note at the top of the template:
It is possible to export with plain LaTeX but an option must be
specified to remove some elements (e.g. JSS logo).

tufte-{book,handout}: set default output to pdf2

If the 'nols' option were set, the files could be output with plain
LaTeX as well; but because letterspacing is an important part of the
Tufte design, letterspacing is preserved and pdf2 is set as the
default output format. This is now explained in a note.

hu/Tutorial.lyx: do not generate bookmarks

By not generating bookmarks, compilation with plain LaTeX works.

Thanks to Jürgen for the solution.

Fix a few typos in French docs

installer: support to spell-check Serbian(Latin)

- also update the Serbian(Cyrillic) dictionary

languages: distinguish between Serbian scripts

- fixes bug #8628

Fix another faulty signature