Scott Kostyshak
82705a0e8a
Fix typo in comment
2017-03-18 11:36:00 -04:00
Guillaume Munch
340d747fff
Clean up
...
Only keep one dynamic-cast. This fixes coverity warnings.
2017-03-18 15:09:07 +01:00
Jean-Marc Lasgouttes
5d29527916
Output debug message when creating bad length
...
Coverity is right to say that we should check the return value.
2017-03-14 15:20:09 +01:00
Jean-Marc Lasgouttes
c6213b1b20
Another forgotten unreachable `return' that is not needed
...
The compilers we use now do not care, but coverity complains.
2017-03-14 14:46:02 +01:00
Jean-Marc Lasgouttes
7a124ab861
Fixup commit 50060053
...
Fix two instances where theFormats() was used instead of a local parameter.
Fix layout test.
2017-03-14 14:26:26 +01:00
Jean-Marc Lasgouttes
7d4b8b7606
Only accept non-negative lyxscale parameters
...
Since lyxscale is unsigned, a negative value would lead to a very
large positive value.
Spotted by coverity.
2017-03-14 14:25:08 +01:00
Richard Heck
50060053e3
Move the global formats and system_formats variables into the
...
LyX singleton.
Mostly, this is very boring, but it might be good if someone would
check what I did about the dummy implementation in tex2lyx.
2017-03-13 23:40:29 -04:00
Jean-Marc Lasgouttes
591f5780e4
Use FileName::lastModified() intead of stat() in DepTable
...
This patch removes explicit calls to stat() in the DepTable code. The
original motivation was to get read of a stat() invokation where the
error condition was not checked (spotted by coverity).
2017-03-13 17:06:42 +01:00
Jean-Marc Lasgouttes
b8c0891427
Remove optimization that avoids drawing some math inset cells.
...
It is not clear that the optimization is a gain, but it is clear that
it has a cost: some elements of formulas will not be present in the
position cache.
Fixes bug #10520 .
2017-03-13 11:44:48 +01:00
Jean-Marc Lasgouttes
a69bae0c99
Make InsetMathNest::editXY more robust
...
If no reasonable cell has been found to put the cursor in (maybe
because the cell exist but is not in the cache), exit gracefully.
Fixes bug #10520
2017-03-13 11:44:22 +01:00
Jean-Marc Lasgouttes
a2bfe0042d
Do not use the global variable `formats' in Formats class
...
Using this global variable in the class is clearly a bug. If we wanted
to avoid doing it by mistake, the variable should be moved to the LyX
singleton.
Also add an underscore to the private variable formatlist_.
2017-03-13 11:39:26 +01:00
Richard Heck
322331c53a
Fix problem noted at bug #10582 .
...
When we have a name with more than two parts, but no "von",
it was coming out as, e.g.:
Obama, Barack Hussain Obama
i.e., with the last name appearing twice.
Also adds a check for names without spaces, which would have given:
Pele, Pele
This was not the original issue at #10582 , so that bug is still
open (though I cannot reproduce it).
2017-03-12 16:43:15 -04:00
Richard Heck
8414c38e4b
Whitespace and const'ing.
2017-03-12 16:27:36 -04:00
Guillaume Munch
797bb2df14
Fix compilation with 5 <= qt < 5.4
2017-03-11 22:22:27 +01:00
Jean-Marc Lasgouttes
5d8918f68f
Compilation fix
...
It is actually better to compile code before pushing changes...
2017-03-11 20:51:39 +01:00
Jean-Marc Lasgouttes
d6d412c49e
Limit the size of strings we read from RCS
...
This calms down Coverity which fears than an evil RCS repository would break LyX.
2017-03-11 20:46:32 +01:00
Scott Kostyshak
3db74fa92d
When catching exception, give error on STDERR
...
The error is reported in the GUI, but for the terminal we must write
explicitly to STDERR.
For a specific example of this type of exception, see the previous
commit, 72c3dbd
.
2017-03-11 02:56:31 -05:00
Scott Kostyshak
72c3dbd02c
Change a warning to an error if missing include
...
If an included file is not found, it seems impossible that the final
output would be correct. Before this commit, when exporting on the
command line, LyX would create the PDF and exit with a zero error
code so unless the user was monitoring the terminal output, they
would not know there is a problem.
This change is consistent with 1a374a93
.
2017-03-11 02:51:49 -05:00
Guillaume Munch
24f68aff8d
Nonsense for whoever insists on using gcc4.6 & qt4.8 in 2017
2017-03-11 00:50:57 +01:00
Guillaume Munch
f96d7a8b2c
Remove legacy FileMonitor
2017-03-11 00:50:57 +01:00
Guillaume Munch
a4fae021f2
Replace FileMonitor with FileMonitor2 in RenderPreview
2017-03-11 00:50:57 +01:00
Guillaume Munch
e3a1396813
Replace FileMonitor with FileMonitor2 in GraphicsCacheItem
...
Remove dead code
2017-03-11 00:50:57 +01:00
Guillaume Munch
22ce6636ef
Clarify dialog
2017-03-11 00:50:57 +01:00
Guillaume Munch
0cb85e9206
Disable edition when external modifications are detected
...
Introduce a distinction between being read-only and having the read-only flag. A
buffer is read-only if either it has been externally modified or if it has the
read-only flag.
2017-03-11 00:50:57 +01:00
Guillaume Munch
b6b56d8518
Notification of external modification inspired by gedit
2017-03-11 00:50:57 +01:00
Guillaume Munch
299242bb16
New LFUN buffer-external-modification-clear
2017-03-11 00:50:57 +01:00
Guillaume Munch
656bc46892
Clean-up
2017-03-11 00:50:57 +01:00
Guillaume Munch
1e57fd12b1
Display externally modified status
2017-03-11 00:50:57 +01:00
Guillaume Munch
7c123507a4
Whitespace
2017-03-11 00:50:57 +01:00
Guillaume Munch
cf3c035266
Implement real-time detection of external modifications
2017-03-11 00:50:57 +01:00
Guillaume Munch
caa54e80ee
Implement FileMonitor as a wrapper for QFileSystemWatcher
...
The new file monitor supports both boost and qt signals. It is implemented in a
ressource-safe way.
2017-03-11 00:50:57 +01:00
Guillaume Munch
22edb3df96
Preventive fix inspired from ~
2017-03-11 00:50:57 +01:00
Guillaume Munch
b034389e6d
Amend 63be456c8
...
Fix crash when clicking on math previews
2017-03-11 00:50:57 +01:00
Guillaume Munch
387ac78088
Add option to ignore a parameter in InsetCommandParams
...
It will not be saved to the file and will always be equal to a given default
value.
2017-03-11 00:50:57 +01:00
Guillaume Munch
3dec5826da
buffer-export default
2017-03-11 00:50:57 +01:00
Jean-Marc Lasgouttes
9a013637bb
Experiment: limit size of strings read from lib/symbols
...
Coverity complains that we might read strings that are arbitrary
large, and that this can be a security issue. This is a problem in
particular, when we feed these strings to from_utf8(), which coverity
flags as dangerous for some reason.
The best solution would be IMO to model from_utf8() properly, but I do
not know how to do that. Here I try a different solution, where I
cannot read a string larger than 64k from the file.
Let's see whether this removes part of coverity warnings.
2017-03-10 16:29:09 +01:00
Jean-Marc Lasgouttes
0f220503ff
Small cleanup to LayoutBox and CategorizedCombo
...
Coverity does not find it obvious that p is never negative. Normally
it is the case (because the items have been filtered), but it is
better to play safe.
2017-03-10 15:57:31 +01:00
Jean-Marc Lasgouttes
e2f2915f8e
Be careful about unparsable bibtex years
...
Handle gracefully the case where the regex that parses a year fails.
This is a fixup to ba171930
(spotted by coverity).
2017-03-10 10:35:03 +01:00
Jean-Marc Lasgouttes
4b8e5a9d53
Another unreachable `return' that is not needed
...
The compilers we use now do not care, but coverity complains.
2017-03-09 15:12:03 +01:00
Jean-Marc Lasgouttes
ce0c11ba4e
Remove unreachable return statements
...
They were needed long ago to avoid gcc warnings, but now all they do
is create coverity warnings.
2017-03-09 15:05:33 +01:00
Jean-Marc Lasgouttes
5a62e37add
Handle the case where doc_bv is null
...
It might only happens in specific cases where no file is open and for
some reason the new file cannot be opened.
2017-03-09 14:41:27 +01:00
Jean-Marc Lasgouttes
f7ab893def
Avoid dereferencing an iterator at the end
...
Found by coverity, could lead to a crash with badly formed kmap file.
2017-03-09 14:26:17 +01:00
Richard Heck
4569010b66
Add a comment.
2017-03-08 17:34:07 -05:00
Jean-Marc Lasgouttes
dee0ea0c21
Make a test clearer
...
This should hopefully please coverity now. For some reason, the
annotation did not work. Should it be in lower case?
2017-03-08 17:03:48 +01:00
Jean-Marc Lasgouttes
000dcb8672
Make a false positive explicit
...
Pleases coverity
2017-03-08 16:50:03 +01:00
Jean-Marc Lasgouttes
2d86b2debb
Fix missing initialization
...
This was already fixed after coverity report at bad511f3fe
,
but this part was missing.
2017-03-08 16:41:42 +01:00
Jean-Marc Lasgouttes
f0ab93c4b8
Initialize members spotted by coverity
2017-03-08 16:37:53 +01:00
Jean-Marc Lasgouttes
ed15c31bf8
Make a dummier LyXRC for tex2lyx
...
The old version made coverity complain about uninitialized members.
The new one is the same as in support/tests/dummy_impl.cpp.
2017-03-08 16:37:53 +01:00
Scott Kostyshak
37290fdb61
Describe the use of '*' in -e/-E CL options
...
The use of '*' in buffer-export was implemented at 494ce664
.
2017-03-08 00:36:28 -05:00
Scott Kostyshak
24ad64c406
Implement '*' in "buffer-export <FORMAT> <DEST>"
...
For the variant of buffer-export that allows a format and a
filename, the substitution must be done after the argument is split.
Related to 494ce664
.
2017-03-08 00:36:28 -05:00