anne/lyx_mirror
1
0
Fork 0
mirror of https://git.lyx.org/repos/lyx.git synced 2024-11-06 11:23:45 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity
46,952 Commits 31 Branches 172 Tags 616 MiB
96c294591c
Commit Graph

29525 Commits

Author SHA1 Message Date
Jean-Marc Lasgouttes
65a014821e Remove unused #include 2024-10-06 20:56:02 +02:00
Juergen Spitzmueller
a16e4b206a Protect \inputencoding in i18npreamble 
as it might end up in a moving argument (\lstlistlistingname in book
classes).

See https://marc.info/?l=lyx-devel&m=172812937932740
 2024-10-06 16:23:05 +02:00
Juergen Spitzmueller
f048b2eeed Fix group box width in Document > Settings > Float Settings 
Also fix typo in widget name
 2024-10-06 15:49:24 +02:00
Jean-Marc Lasgouttes
4d89b2903b Fix up 6c20e5db: let PARAGRAPH_GOTO use SCROLL_TOP again 
As said, this was an experiment. It did not seem to stick.
 2024-10-05 15:30:22 +02:00
Jean-Marc Lasgouttes
09684899d6 Clean up after removal of strfwd.h 
Remove autoconf tests for whether std::string uses COW, since this is
forbidden by C++11 standard.

Forbid the use of gcc 4.9 (which still uses COW).

Remove code in debug.h that added forward declarations for LLVM's
libc++, since we do not do forward declarations anymore.

Remove configure test for libc++.
 2024-10-05 15:15:21 +02:00
Juergen Spitzmueller
1f7e8d3ffe Adhere to semantic background color with default branches 
This fixes color mismatched when "use system colors" is checked

See https://marc.info/?l=lyx-users&m=172797086718347
 2024-10-05 14:15:19 +02:00
Pavel Sanda
2facf6eecc Merged compilation does not exist anymore. 2024-10-04 20:53:26 +02:00
Pavel Sanda
b8f1f113f3 Remove trivstring. 
Needed in times when std:string was not thread-safe,
but should be ok nowadays.

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=21334
https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg223068.html
 2024-10-04 20:49:55 +02:00
Pavel Sanda
9ccda98e6e Remove strfwd.h. 
Will break cygwin due to trivstring header.

https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg223056.html
 2024-10-04 20:49:33 +02:00
Jean-Marc Lasgouttes
69d8435a7f Avoid copying vector needlessly 
The getLabel method will not consider more than 10 citation keys.
Instead of removing elements from the keys vector, this commit adapts
the for loop to skip unwanted elements.

This allows to pass the keys vector by const reference.

Spotted by Coverity scan.
 2024-10-03 10:18:24 +02:00
Jean-Marc Lasgouttes
bc624cc780 Do not pass a vector by value to getArgInset 
Instead of passing the vector by value so that stuff can be added to
it in the function, add required elements of arguments to a second vector.

While we are at it, simplify the code a bit by using a range-for and
auto keyword.

Spotted by Coverity scan
 2024-10-01 13:06:37 +02:00
Jean-Marc Lasgouttes
54b6846941 Pass parameters as const reference 
Spotted by Coverity scan.
 2024-10-01 11:12:14 +02:00
Pavel Sanda
b39cc9ed89 Not needed anymore. 2024-09-30 20:11:23 +02:00
Pavel Sanda
601abfe929 PDF options: reflect in UI mutually exclusive colorlinks and frames. 
https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg222961.html
 2024-09-30 20:09:35 +02:00
Jean-Marc Lasgouttes
1d64dbf543 Modify string list in place instead of copying it 
Spotted by Coverity scan.
 2024-09-30 11:20:48 +02:00
Koji Yokota
a086804eb4 Fix IM window position for a new window (#13094) 2024-09-30 09:00:56 +09:00
Thibaut Cuvelier
75f467fa13 MathML: use Core for XHTML, 3 for DocBook. 
In a near future, this should rather be controlled by a user-visible parameter.
 2024-09-28 16:50:29 +02:00
Thibaut Cuvelier
790c153737 Use proper minus sign character in MathML Core. 
Fixes bug https://www.lyx.org/trac/ticket/13067.

Based on the patch in the ticket above by Jean-Marc Lasgouttes.
 2024-09-28 16:50:29 +02:00
Thibaut Cuvelier
ca5d5a0b1a amend 2024-09-28 16:50:29 +02:00
Thibaut Cuvelier
a06528ee2a Fix switch indentation. 2024-09-28 16:50:29 +02:00
Thibaut Cuvelier
cb88a362ea MathML: add a version enum. 2024-09-28 16:50:29 +02:00
Jean-Marc Lasgouttes
9fc19badab Streamline use of map<col-type, bool> 
Typically, to check whether an element is present and set to true, instead of
  foo.find(c) != foo.end() && foo.find(c)->second
one uses
  foo[c]

The map<> code creates elements automatically when they do not exist
and zero-initializes them.

This avoids puzzlement of Coverity scan when seeing that find() result
is dereferenced without checking that it is not equal to end().
Moreover, this makes the code much much clearer. I still do not
understand it, though ;)
 2024-09-26 15:31:28 +02:00
Juergen Spitzmueller
112c2c3d0e Factor out (signed) glueLenghValidator 2024-09-25 10:22:36 +02:00
Juergen Spitzmueller
2244ac3dee Allow negative offset in GuiLine (#13102) 2024-09-25 09:53:00 +02:00
Juergen Spitzmueller
126cbec5e7 Re-focus workarea after some outline context-menu actions (see #1624) 
Thanks Pavel for the main idea
 2024-09-24 08:47:55 +02:00
Jean-Marc Lasgouttes
194b53ef9d QFontMetrics::xHeight() is not always the ascent of 'x' 
Use the real ascent of glyph 'x' as LyX 2.3.x did.
It can make a difference at least on Windows with many fonts.

Fixes bug #13100.
 2024-09-23 21:11:49 +02:00
Juergen Spitzmueller
d43e82a5a7 Only \protect labels in \thanks notes 
See https://marc.info/?l=lyx-users&m=136394387505489
 2024-09-21 11:30:08 +02:00
Juergen Spitzmueller
124f441a00 GuiBibTeX: introduce way to toggle between absolute and relative local paths (#13097) 2024-09-21 07:30:55 +02:00
Pavel Sanda
b7e4cc9a37 Header not needed anymore 2024-09-20 21:33:41 +02:00
Jean-Marc Lasgouttes
77e4092b99 Move assertion at the right position 
It would wrongly trigger when doing a multi-cell selection.
 2024-09-19 23:18:47 +02:00
Juergen Spitzmueller
f5306b7054 Fix outliner tree depth (#13098) 2024-09-19 10:00:39 +02:00
Pavel Sanda
9d9f13ff83 Move header where needed. 2024-09-18 21:25:18 +02:00
Pavel Sanda
c87b2d5fc2 Header not needed anymore 2024-09-17 19:51:35 +02:00
Jean-Marc Lasgouttes
928f9514a8 More places where DocumentClassPtr should not be copied 
Spotted by COverity scan.
 2024-09-14 20:40:53 +02:00
Jean-Marc Lasgouttes
8c76a9fc3d Pass sigPtr object as const reference too 
Spotted by Coverity scan.
 2024-09-14 20:36:04 +02:00
Jean-Marc Lasgouttes
980f91d12e Pass SocketCallback object as const reference 
Spotted by Coverity scan.
 2024-09-14 20:19:34 +02:00
Juergen Spitzmueller
30f170ea26 Revert "use std::move() instead of copying when value is no longer used" 
This reverts commit e685d31f5a.
 2024-09-14 10:59:13 +02:00
Juergen Spitzmueller
3a1c8897b1 Revert "Fix compiler warning" 
This reverts commit 0a75854038.
 2024-09-14 10:59:08 +02:00
Jean-Marc Lasgouttes
375aba7b5d Transform CoordCache::check/checkDim in ASSERT_DIM/POS macros 
This makes it more obvious to the reader that these are actually
assertions, and should help Coverity scan too.
 2024-09-13 15:44:33 +02:00
Jean-Marc Lasgouttes
3af0bad22a Pass shared_ptr<> arguments by const reference 
These arguments are not expensive to copy. However, it makes sense to
pass such pointers by const reference when they are just intended for
reading.

Some reading about this issue is here:
https://www.internalpointers.com/post/move-smart-pointers-and-out-functions-modern-c

Fixes some Coverity scan defects.
 2024-09-13 14:54:43 +02:00
Jean-Marc Lasgouttes
58ee4c9ec7 Pass DocumentClassConstPtr arguments by const reference 
This arguments are shared_ptr objects, so they are not expensive to
copy. However, it makes sense to pass such pointers by const reference
when they are just intended for reading.

Some reading about this issue is here:
https://www.internalpointers.com/post/move-smart-pointers-and-out-functions-modern-c

Fixes some Coverity scan defects.
 2024-09-13 14:16:21 +02:00
Jean-Marc Lasgouttes
3649cea9e5 Make two parameters const references 2024-09-13 11:27:00 +02:00
Jean-Marc Lasgouttes
ffca6730d6 Fix uninitialized variable 
Spotted by Coverity scan.
 2024-09-13 11:21:19 +02:00
Jean-Marc Lasgouttes
9f40eaee15 Do not use rand() to set a BranchList id 
Use a simple counting instead, beecause Coverity complains that rand()
is not safe, and counting is siimpler anyway.
 2024-09-13 11:09:30 +02:00
Jean-Marc Lasgouttes
9b3c28178c Make string parameter a const reference 
Spotted by Coverity scan.
 2024-09-13 10:27:24 +02:00
Jean-Marc Lasgouttes
8e181e992d Avoid a couple of copies. 
Spotted by Coverity scan.
 2024-09-12 18:33:45 +02:00
Jean-Marc Lasgouttes
7a99ef7fbb Make docstring parameters const references 
Spotted by Coverity scan.
 2024-09-12 18:28:26 +02:00
Jean-Marc Lasgouttes
b743870d71 Graph::getReachable(): pass a parameter by const reference 
Spotted by coverity.
 2024-09-12 18:20:43 +02:00
Jean-Marc Lasgouttes
b1bbf64a28 Avoid a copy 
Spotted by Coverity scan.
 2024-09-12 18:05:50 +02:00
Jean-Marc Lasgouttes
cc40330e00 Avoid a couple of copies 
Spotted by Coverity scan.
 2024-09-12 17:59:41 +02:00