Re: hi

2024-10-08 Thread Jean-Marc Lasgouttes
Le 08/10/2024 à 08:47, Ali Roshdi a écrit : Hi I want to register LyX and I am copying ZWJ from Excel and Word MS but they can not be copied to LyX?! Hi Ali, First, what do you mean by “register”? Do you want to subscribe to the users mailing list? Or to the developers mailing list? Second

Re: hi

2024-10-08 Thread Jean-Marc Lasgouttes
Le 08/10/2024 à 08:47, Ali Roshdi a écrit : Hi I want to register LyX and I am copying ZWJ from Excel and Word MS but they can not be copied to LyX?! Hi Ali, First, what do you mean by “register”? Do you want to subscribe to the users mailing list? Or to the developers mailing list? Second

Re: hi

2024-10-08 Thread Jean-Marc Lasgouttes
Le 08/10/2024 à 18:04, Jürgen Spitzmüller a écrit : Am Dienstag, dem 08.10.2024 um 11:51 -0400 schrieb Richard Kimberly Heck: What is ZWJ? ZERO WIDTH JOINER (0x200D). It is used to control ligatures especially in Arabic and Indic scripts. We have it commented out in unicodesymbols. I am not s

Re: LyX 2.4.2 Released

2024-10-06 Thread Jean-Marc Lasgouttes
Le 06/10/2024 à 22:17, mkriele--- via lyx-users a écrit : Dear developers I just wanted to drop you a line to tell you how much I appreciate this release. We have been treated to many great releases of Lyx, and 2.4 was really impressive. But for me personally, this release made all the diffe

Re: Big crash while S&R advanced

2024-10-06 Thread Jean-Marc Lasgouttes
Le 06/10/2024 à 09:34, Murat Yildizoglu a écrit : I wanted to signal as a bug but the bug tracker has forgotten me it seems and my login is rejected now. Hi Murat, Let's start with this. Can you send me privately your login name and the associated address (if it is not the one I see here)?

Re: [LyX/master] Add ScrollType BOTTOM and TOGGLE for BufferView::scrollToCursor()

2024-10-05 Thread Jean-Marc Lasgouttes
Le 19/09/2024 à 23:19, Pavel Sanda a écrit : Before I revert that part, did you try to click several times on the toc entry? I personally think that it is a good way to look around cursor position. Yes, I tried. I am not against that functionality, but I would like the first click puts the curs

Re: [LyX/master] Clean up after removal of strfwd.h

2024-10-05 Thread Jean-Marc Lasgouttes
Le 05/10/2024 à 15:15, Jean-Marc Lasgouttes a écrit : commit 09684899d68972ff2dc415218a8411fc7ccc2b69 Author: Jean-Marc Lasgouttes Date: Sat Oct 5 15:04:51 2024 +0200 Clean up after removal of strfwd.h Remove autoconf tests for whether std::string uses COW, since this is

Re: GCC >= 5? (was: [patch] Removal of strfwd.h)

2024-10-04 Thread Jean-Marc Lasgouttes
Le 04/10/2024 à 19:18, Enrico Forestieri a écrit : We should now either get rid of STD_STRING_USES_COW or implement the right check for it. I would get rid of it, since it is required by the standard to not use COW. Our new requirement is "a C++ compiler that _really_ implements C++11" (remem

Re: GCC >= 5? (was: [patch] Removal of strfwd.h)

2024-10-04 Thread Jean-Marc Lasgouttes
Le 04/10/2024 à 17:20, Pavel Sanda a écrit : On Fri, Oct 04, 2024 at 04:46:01PM +0200, Jean-Marc Lasgouttes wrote: I did not read completely your comment in docstring.h. I think we can get rid completely of the names trvstring and trivdocstring soon after this landed. If we are sure that the

Re: GCC >= 5? (was: [patch] Removal of strfwd.h)

2024-10-04 Thread Jean-Marc Lasgouttes
Le 04/10/2024 à 16:36, Pavel Sanda a écrit : On Fri, Oct 04, 2024 at 04:18:54PM +0200, Jean-Marc Lasgouttes wrote: This looks good to me. Note that this whole test program can be removed: I was thinking about it and spared it as you can see it as a test for docstring sanity now. But I can

Re: GCC >= 5? (was: [patch] Removal of strfwd.h)

2024-10-04 Thread Jean-Marc Lasgouttes
Le 04/10/2024 à 15:17, Pavel Sanda a écrit : On Fri, Oct 04, 2024 at 12:51:56PM +0200, Enrico Forestieri wrote: On Fri, Oct 04, 2024 at 11:52:01AM +0200, Pavel Sanda wrote: I am all for ditching trivstring, but it all depends on Enrico's stance, I didn't folow the previous discussion on string

[PATCH] Add move constructor and move assignment operator for, FileName class

2024-10-03 Thread Jean-Marc Lasgouttes
This is my take on this patch from yuriy, after 854c9de8 had to be reverted. See discussion here: https://marc.info/?t=16101843892&r=2&w=2 In the case of move constructor, instead of setting the Private pointer of the moved FileName to nullptr, set it to a pointer to a static FileName::Pri

Re: [patch] Removal of strfwd.h

2024-10-03 Thread Jean-Marc Lasgouttes
Le 03/10/2024 à 23:33, Pavel Sanda a écrit : But I don't like it there as all trivstring stuff should imho go into trivstring.h. Before I was afraid what will break, but as I broke cygwin anyway, maybe it's time to get rid of this from docstring? If we decide that we want at least gcc5, we ca

Re: Repository not accessible ATM

2024-10-03 Thread Jean-Marc Lasgouttes
Le 03/10/2024 à 14:34, Kornel Benko a écrit : About 13:38 CEST the access was OK, now I get (after 5 mins) ssh: connect to host git.lyx.org port 22: Connection timed out fatal: Could not read from remote repository. Please make sure you have the correct access rights and the repository exists.

Re: When to use strfwd.h

2024-10-01 Thread Jean-Marc Lasgouttes
Le 01/10/2024 à 18:18, Pavel Sanda a écrit : I could produce patch which ditches strfwd and let Enrico/Eugene test it? From what I see it might do more harm than good in linux nowadys... Go for it. And do not forget to remove and/or from the cpp file. JMarc -- lyx-devel mailing list lyx

Re: When to use strfwd.h

2024-10-01 Thread Jean-Marc Lasgouttes
Le 01/10/2024 à 10:50, Pavel Sanda a écrit : Hi, in GuiToolbar.h we include strfwd.h, clearly because we use std::string there (as introduced by Riki in cc4bfc7f04d). However in many different .h files we include for the same purpouse. I suspect we should be consistent but I am not sure what i

Re: When to use strfwd.h

2024-10-01 Thread Jean-Marc Lasgouttes
Le 01/10/2024 à 12:23, Pavel Sanda a écrit : On Tue, Oct 01, 2024 at 11:12:00AM +0200, Jean-Marc Lasgouttes wrote: Basically, for gcc >=5 and libc++, we include . Unless there is something I do not understand, the header is not really useful IMO. Are you sure about this? I checked bef

Re: New Defects reported by Coverity Scan for LyX

2024-09-30 Thread Jean-Marc Lasgouttes
Le 30/09/2024 à 17:35, Jürgen Spitzmüller a écrit : Am Montag, dem 30.09.2024 um 15:06 +0200 schrieb Jean-Marc Lasgouttes: Thanks for walking me through the testing. It (still) works, so I pushed this. Thanks. I only bothered you with this since I was away from my machine. Now I can re-run

Re: New Defects reported by Coverity Scan for LyX

2024-09-30 Thread Jean-Marc Lasgouttes
Le 30/09/2024 à 09:13, Jürgen Spitzmüller a écrit : * Add a local bib file in the Bib(la)TeX dialog * Select this entry in the selected bibs widget and press "Make Absolute" or "Make Relative" (depending on what the button text displays, which should reflect the current state) * If the path togg

Re: New Defects reported by Coverity Scan for LyX

2024-09-29 Thread Jean-Marc Lasgouttes
5ddd2879f096637de9bdbb7ebaa95ac3 Mon Sep 17 00:00:00 2001 From: Jean-Marc Lasgouttes Date: Sun, 29 Sep 2024 21:45:08 +0200 Subject: [PATCH] Modify string list in place instead of copying it Spotted by Coverity scan. --- src/frontends/qt/GuiBibtex.cpp | 8 ++-- 1 file changed, 2 insertions(+),

Re: Bugs and account request

2024-09-27 Thread Jean-Marc Lasgouttes
. Have a nice weekend, JMarc Le 27/09/2024 à 18:36, Jean-Marc Lasgouttes a écrit : Le 27/09/2024 à 18:29, John Koutas a écrit : Hello, I am writing to request an account to submit bugs for Lyx. In any case, the two issues I have encountered when moving from 2.3 to 2.4.1 (Windows 10 22H2, Qt 6.7.2

Re: Bugs and account request

2024-09-27 Thread Jean-Marc Lasgouttes
Le 27/09/2024 à 18:29, John Koutas a écrit : Hello, I am writing to request an account to submit bugs for Lyx. In any case, the two issues I have encountered when moving from 2.3 to 2.4.1 (Windows 10 22H2, Qt 6.7.2, Python 3.12.6) are the following: Hello John, * Even when "mark end of

Re: Coverity musings

2024-09-26 Thread Jean-Marc Lasgouttes
Le 19/09/2024 à 16:27, Jürgen Spitzmüller a écrit : Am Donnerstag, dem 19.09.2024 um 16:16 +0200 schrieb Jean-Marc Lasgouttes: What I have in mind as a simpler alternative is this (like the last patch, with vector instead of map). Seems to make sense. I have no time to look closer now

Re: [LyX/master] Add ScrollType BOTTOM and TOGGLE for BufferView::scrollToCursor()

2024-09-19 Thread Jean-Marc Lasgouttes
Le 10/09/2024 à 15:56, Pavel Sanda a écrit : On Fri, Sep 06, 2024 at 03:33:24PM +0200, Jean-Marc Lasgouttes wrote: Then, I changed the outline to yield the same effect. I expect some of you to hate it, but discussion is good and this is not the final design. Again, clicking several times on an

Re: Coverity musings

2024-09-19 Thread Jean-Marc Lasgouttes
Le 19/09/2024 à 15:40, Jürgen Spitzmüller a écrit : Am Donnerstag, dem 19.09.2024 um 15:14 +0200 schrieb Jean-Marc Lasgouttes: But why go from a vector<> to a map<>? Frankly: I can't remember. What I have in mind as a simpler alternative is this (like the last patch, wi

Re: Coverity musings

2024-09-19 Thread Jean-Marc Lasgouttes
Le 15/09/2024 à 18:14, Jürgen Spitzmüller a écrit : Am Samstag, dem 14.09.2024 um 20:07 +0200 schrieb Jean-Marc Lasgouttes: Back in history, the structure was not a map but a vecor. It is used to track things like "is there a line below this cell?". I am not sure why  Juergen chang

Re: Coverity musings

2024-09-14 Thread Jean-Marc Lasgouttes
Le 14/09/2024 à 15:40, Pavel Sanda a écrit : Can you explain what that construct works? If I understand correctly we are changing the underlying structure and that does not feel right for "search" procedure. Apart from that if we search extensively the memory will grow as well? First, for the f

Re: Coverity musings

2024-09-14 Thread Jean-Marc Lasgouttes
Le 14/09/2024 à 00:43, Richard Kimberly Heck a écrit : Do you have access to coverity? I think I did, but I do not remember how to access it. The password, etc, is probably saved in Firefox, if you just send me the URL. Have a look here. https://scan.coverity.com/projects/4164 You do have a

Re: [LyX/master] use std::move() instead of copying when value is no longer used

2024-09-14 Thread Jean-Marc Lasgouttes
Thanks. I would have done it otherwise. A good time to recall that "premature optimization is the root of all evil." JMarc Le 14 septembre 2024 11:01:03 GMT+02:00, "Jürgen Spitzmüller" a écrit : >Am Dienstag, dem 10.09.2024 um 15:21 +0200 schrieb Jean-Marc >Lasgou

Re: Coverity musings

2024-09-13 Thread Jean-Marc Lasgouttes
Le 13/09/2024 à 20:44, Richard Kimberly Heck a écrit : * Places where I understand the issue, but fixing requires some work and is a bit scary. Example in the patch below: Jürgen, could you please have a look and tell me why we should not do that? It is too simple, I fear I am missing something

Coverity musings

2024-09-13 Thread Jean-Marc Lasgouttes
am not sure of a proper fix. * Places where I do not even understand what the issue is :( Have a nice weekend. JMarcFrom 213d8088bcbcae78ad735bb46bc8b1fb93e75bca Mon Sep 17 00:00:00 2001 From: Jean-Marc Lasgouttes Date: Fri, 13 Sep 2024 17:10:58 +0200 Subject: [PATCH] Streamline use of map Typ

Re: [LyX/master] Do not use rand() to set a BranchList id

2024-09-13 Thread Jean-Marc Lasgouttes
Le 13/09/2024 à 11:37, Pavel Sanda a écrit : On Fri, Sep 13, 2024 at 11:14:13AM +0200, Jean-Marc Lasgouttes wrote: Le 13/09/2024 ?? 11:11, Jean-Marc Lasgouttes a écrit : commit 9f40eaee15d53ce418a728d3ff3716541d35775f Author: Jean-Marc Lasgouttes Date: Fri Sep 13 11:07:05 2024 +0200

Re: [LyX/master] Do not use rand() to set a BranchList id

2024-09-13 Thread Jean-Marc Lasgouttes
Le 13/09/2024 à 11:11, Jean-Marc Lasgouttes a écrit : commit 9f40eaee15d53ce418a728d3ff3716541d35775f Author: Jean-Marc Lasgouttes Date: Fri Sep 13 11:07:05 2024 +0200 Do not use rand() to set a BranchList id Use a simple counting instead, beecause Coverity complains that

Re: Registration

2024-09-13 Thread Jean-Marc Lasgouttes
Le 13/09/2024 à 10:50, Pavel Sanda a écrit : On Fri, Sep 13, 2024 at 10:44:55AM +0200, Pavel Sanda wrote: On Thu, Sep 12, 2024 at 11:32:55AM +0200, Daniel wrote: Are there any alternatives in the pipe? Not really, but I can improve the description to be more straightforward. Just did that.

Re: [LyX/master] Do not handle special case where buffer == NULL

2024-09-11 Thread Jean-Marc Lasgouttes
Le 9 septembre 2024 19:01:09 GMT+02:00, "José Matos" a écrit : >On Mon, 2024-09-09 at 17:54 +0200, Jean-Marc Lasgouttes wrote: >> string const arg = (type.empty()) ? move(label) : move(label) + " " >> + type; > >I can understand the first move. Tha

Re: [LyX/master] use std::move() instead of copying when value is no longer used

2024-09-11 Thread Jean-Marc Lasgouttes
Le 07/09/2024 à 13:42, Jürgen Spitzmüller a écrit : Am Samstag, dem 07.09.2024 um 13:22 +0200 schrieb Jean-Marc Lasgouttes: I have been resisting to making these changes because someone may change the code and use the variable afterwards. But I may be wrong after all. I am not sure at all. I

Re: [LyX/master] use std::move() instead of copying when value is no longer used

2024-09-10 Thread Jean-Marc Lasgouttes
Le 10/09/2024 à 15:01, Pavel Sanda a écrit : I find that std::move construct make the readbility worse for me. Not seeing it on the left side of the assignment makes it easy to overlook that the variable might be actually empty after move and use it in the subsequent code later. Unless we are op

Re: [LyX/master] Initialize client's name in main()

2024-09-10 Thread Jean-Marc Lasgouttes
Le 10/09/2024 à 00:04, Richard Kimberly Heck a écrit : On 9/9/24 11:58 AM, Jean-Marc Lasgouttes wrote: Le 09/09/2024 à 15:33, Jean-Marc Lasgouttes a écrit : commit 3d323412ad5b4d47774de43f985202f20cabd156 Author: Jean-Marc Lasgouttes Date:   Mon Sep 9 15:29:39 2024 +0200 Initialize

Re: [LyX/master] Do not forget to reset stream after std::hex

2024-09-09 Thread Jean-Marc Lasgouttes
Le 09/09/2024 à 17:51, Jean-Marc Lasgouttes a écrit : commit f4c02d670b60a56bc6fb4a49e201c7134af1011b Author: Jean-Marc Lasgouttes Date: Mon Sep 9 17:49:20 2024 +0200 Do not forget to reset stream after std::hex Riki, this is in stable to. OK to backport? JMarc Spotted by

Re: [LyX/master] Do not handle special case where buffer == NULL

2024-09-09 Thread Jean-Marc Lasgouttes
Le 09/09/2024 à 17:35, Scott Kostyshak a écrit : I see, good to know! Thanks to you and JMarc for taking on that seemingly never-ending task. The good thing is that we caught a few real bugs. It is not always coverity being annoying. JMarc -- lyx-devel mailing list lyx-devel@lists.lyx.org h

Re: [LyX/master] Do not handle special case where buffer == NULL

2024-09-09 Thread Jean-Marc Lasgouttes
Le 09/09/2024 à 16:47, Jürgen Spitzmüller a écrit : Am Montag, dem 09.09.2024 um 16:31 +0200 schrieb Scott Kostyshak: Just out of curiosity (no I don't want to receive the scan results ;)), are most of these from Coverity adding new features that detect new things? Or were they there before, and

Re: label whitespaces no more

2024-09-08 Thread Jean-Marc Lasgouttes
Le 08/09/2024 à 19:09, Jürgen Spitzmüller a écrit : Am Sonntag, dem 08.09.2024 um 12:18 -0400 schrieb Richard Kimberly Heck: That was my thought, as well. Simple and effective. Good, I committed to master. Note that if you cherry-pick, you both need both bfd855747ab (the indentation fix) and 0

Re: [LyX/master] use std::move() instead of copying when value is no longer used

2024-09-07 Thread Jean-Marc Lasgouttes
Le 07/09/2024 à 12:52, Juergen Spitzmueller a écrit : commit e685d31f5aadadd9c84c80a604818d49f5853dbf Author: Juergen Spitzmueller Date: Sat Sep 7 12:50:27 2024 +0200 use std::move() instead of copying when value is no longer used This is mainly to shut down coverity which now

Re: [LyX/master] Add ScrollType BOTTOM and TOGGLE for BufferView::scrollToCursor()

2024-09-06 Thread Jean-Marc Lasgouttes
Le 06/09/2024 à 14:56, Jean-Marc Lasgouttes a écrit : commit 6c20e5db7db662239e8c2bd47fe28d18b27bd729 Author: Jean-Marc Lasgouttes Date: Thu Sep 5 22:05:38 2024 +0200 Add ScrollType BOTTOM and TOGGLE for BufferView::scrollToCursor() BOTTOM shows the paragraph containing the

Re: program listing crash

2024-08-31 Thread Jean-Marc Lasgouttes
Le 30/08/2024 à 19:48, Richard Kimberly Heck a écrit : Riki, this is obviously needed in stable too. In this setting, there is a separate issue that selection drawing fails. All in all, I propose to push the two patches below. OK. It is in, thanks. JMarc -- lyx-devel mailing list lyx-devel@

Re: [LyX/master] Do not run updateMacros if the buffer has not changed

2024-08-30 Thread Jean-Marc Lasgouttes
Le 30/08/2024 à 17:40, Scott Kostyshak a écrit : Great, thanks! Although from what I understand it has zero effect ;). Indeed. Even if we were to use all possible integers too soon, nothing bad would happen :) JMarc -- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mai

Re: program listing crash

2024-08-30 Thread Jean-Marc Lasgouttes
46e697288d Mon Sep 17 00:00:00 2001 From: Jean-Marc Lasgouttes Date: Fri, 30 Aug 2024 16:56:16 +0200 Subject: [PATCH 1/2] Fix update of cursor in tab-delete when there is a selection The position of the cursor should not be decreased if it is already at the start of the paragraph. This can lead

Re: [LyX/master] Do not run updateMacros if the buffer has not changed

2024-08-30 Thread Jean-Marc Lasgouttes
Le 30/08/2024 à 14:27, Scott Kostyshak a écrit : I have a question only out of curiosity (it doesn't have any practical consequence): Also, is id_ ever updated by just 1 in updateId()? When I put debug code to print id_, when I type I see: id_: 0 id_: 2 id_: 4 You are right, of course. I kn

Re: [for enrico] Question: do we still need that?

2024-08-29 Thread Jean-Marc Lasgouttes
Le 29/08/2024 à 13:51, Enrico Forestieri a écrit : What I did not do is remove the hand-made parsing of geometry that is used for window-new lfun only on Windows (why?). Can this go away? That code was necessary for Qt4, where -qwindowgeometry was not available. It was necessary on Windows bec

Re: [for enrico] Question: do we still need that?

2024-08-29 Thread Jean-Marc Lasgouttes
HAVE_XCB_XCB_H HAVE_LIXCB if they are available.From f2efed661bde9e03baf8269ee1f302c7489a162e Mon Sep 17 00:00:00 2001 From: Jean-Marc Lasgouttes Date: Thu, 29 Aug 2024 11:10:31 +0200 Subject: [PATCH 1/2] Streamline -geometry argument support --- src/LyX.cpp | 23

Re: [for enrico] Question: do we still need that?

2024-08-28 Thread Jean-Marc Lasgouttes
Le 28/08/2024 à 15:54, Jean-Marc Lasgouttes a écrit : In createAppplication, we have: #if !defined(QPA_XCB) // prune -geometry argument(s) by shifting // the following ones 2 places down. for (int i = 0 ; i < argc ; ++i) {     if (strcmp(argv[i], "-geometr

[for enrico] Question: do we still need that?

2024-08-28 Thread Jean-Marc Lasgouttes
In createAppplication, we have: #if !defined(QPA_XCB) // prune -geometry argument(s) by shifting // the following ones 2 places down. for (int i = 0 ; i < argc ; ++i) { if (strcmp(argv[i], "-geometry") == 0) { int const remove = (i+1

Re: Regression in 2.4 - external rewrite causes buffer to be marked as modified

2024-08-19 Thread Jean-Marc Lasgouttes
Hi, I do not remember such a deliberate changel, but I may have missed it (or we'll find out that I did it on purpose and forgot about it 😁). JMarc Le 19 août 2024 11:04:16 GMT+02:00, Pavel Sanda a écrit : >Hi, > >can't remember whether this was intention but I find the new behaviour very >u

Re: [LyX/master] Implement Paragraph::getLabel()

2024-07-29 Thread Jean-Marc Lasgouttes
Le 29 juillet 2024 09:02:49 GMT+02:00, "Jürgen Spitzmüller" a écrit : >Am Montag, dem 29.07.2024 um 08:29 +0200 schrieb Jürgen Spitzmüller: >> Am Sonntag, dem 28.07.2024 um 19:39 +0200 schrieb Jean-Marc >> Lasgouttes: >> > Usually, when we use getLabel,

Re: [LyX/master] Implement Paragraph::getLabel()

2024-07-28 Thread Jean-Marc Lasgouttes
Le 28 juillet 2024 16:00:56 GMT+02:00, Juergen Spitzmueller a écrit : >commit a303600b544d9f4c82f2dee77616f821d0051854 >Author: Juergen Spitzmueller >Date: Sun Jul 28 15:52:19 2024 +0200 > >Implement Paragraph::getLabel() > >This function returns the first label (as string) if th

Re: [RFC] Make scrolling-by-selection smoother

2024-07-26 Thread Jean-Marc Lasgouttes
But is it faster when the mouse is below the window, then? JMarc Le 25 juillet 2024 21:14:36 GMT+02:00, Scott Kostyshak a écrit : >On Thu, Jul 25, 2024 at 08:48:49PM GMT, Jean-Marc Lasgouttes wrote: >> You are in full screen, right ? > >No, but by default I don't have

Re: [RFC] Make scrolling-by-selection smoother

2024-07-25 Thread Jean-Marc Lasgouttes
You are in full screen, right ? JMarc Le 25 juillet 2024 20:00:15 GMT+02:00, Scott Kostyshak a écrit : >On Thu, Jul 25, 2024 at 06:45:58PM GMT, Jean-Marc Lasgouttes wrote: >> >> > To me it seems that scrolling down (if I hold and drag the mouse) is >> > much slowe

Re: [RFC] Make scrolling-by-selection smoother

2024-07-25 Thread Jean-Marc Lasgouttes
Le 25/07/2024 à 18:42, Scott Kostyshak a écrit : On Mon, Jul 22, 2024 at 10:00:46PM GMT, Jean-Marc Lasgouttes wrote: Le 22/07/2024 à 21:49, Pavel Sanda a écrit : On Sat, Jul 20, 2024 at 09:22:02PM +0200, Jean-Marc Lasgouttes wrote: So, especially if you have a slow-ish computer, please test

Re: [LyX/master] mode convertLaTeXCommands from BiblioInfo to Encodings

2024-07-25 Thread Jean-Marc Lasgouttes
Le 25/07/2024 à 15:20, Jürgen Spitzmüller a écrit : Am Donnerstag, dem 25.07.2024 um 11:30 +0200 schrieb Jean-Marc Lasgouttes: Did I already say today that I hate updateMacros and have no idea of what it does ? Just wanted to mention that this seems to cure it: Smart man. Of course, it

Re: [LyX/master] mode convertLaTeXCommands from BiblioInfo to Encodings

2024-07-25 Thread Jean-Marc Lasgouttes
isect. JMarc Bisect points me to: commit 999fb37ebb1aecfda9889fa6a2b2fbb15133f2aa Author: Jean-Marc Lasgouttes Date: Sat Jul 20 22:31:34 2024 +0200 Do not run updateMacros if the buffer has not changed Each buffer now has an id which is increased when it is marked dirty

Re: [LyX/master] Do not run updateMacros if the buffer has not changed

2024-07-24 Thread Jean-Marc Lasgouttes
Le 25/07/2024 à 00:43, Richard Kimberly Heck a écrit : On 7/24/24 12:36 PM, Jean-Marc Lasgouttes wrote: Le 24/07/2024 à 18:31, Jean-Marc Lasgouttes a écrit : commit 999fb37ebb1aecfda9889fa6a2b2fbb15133f2aa Author: Jean-Marc Lasgouttes Date:   Sat Jul 20 22:31:34 2024 +0200 Do not run

Re: [PATCH] WIP: rewrite statistics code

2024-07-24 Thread Jean-Marc Lasgouttes
Le 24/07/2024 à 22:48, Pavel Sanda a écrit : I have difficulty to see any visible difference when it comes to the cursor speed in UG after your last patch (detecting changes in buffer before calling updatemacros), i.e. the difference seems to be within rounding error. The updateMacros probably i

Re: [PATCH] WIP: rewrite statistics code

2024-07-24 Thread Jean-Marc Lasgouttes
Le 24/07/2024 à 01:57, Pavel Sanda a écrit : On top of that this speed up was good enough that I do not see visually noticeable hiccups while moving with the cursor in User Guide anymore - but that's also due to the fact that cursor moving is still somewhat slow on fast repeat rates compared to o

Re: [PATCH] WIP: rewrite statistics code

2024-07-24 Thread Jean-Marc Lasgouttes
Le 24/07/2024 à 19:28, Scott Kostyshak a écrit : /home/scott/lyxbuilds/master-master/repo/src/insets/InsetTabular.h:1120:7: error: 'updateStatistics' overrides a member function but is not marked 'override' [-Werror,-Winconsistent-missing-override] void updateStatistics(Statistics & st

Re: [LyX/master] Do not run updateMacros if the buffer has not changed

2024-07-24 Thread Jean-Marc Lasgouttes
Le 24/07/2024 à 18:31, Jean-Marc Lasgouttes a écrit : commit 999fb37ebb1aecfda9889fa6a2b2fbb15133f2aa Author: Jean-Marc Lasgouttes Date: Sat Jul 20 22:31:34 2024 +0200 Do not run updateMacros if the buffer has not changed Each buffer now has an id which is increased when it

Re: [LyX/master] Fixup 216a6fb348: close when quitting text mode

2024-07-24 Thread Jean-Marc Lasgouttes
Le 24/07/2024 à 17:46, Richard Kimberly Heck a écrit : Riki, this is candidate for branch. Any and all fixes for this are fine. Thanks, everything is in now. JMarc -- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mailman/listinfo/lyx-devel

Re: [LyX/master] Fixup 216a6fb3: fix broken xml syntax

2024-07-24 Thread Jean-Marc Lasgouttes
Le 24/07/2024 à 17:45, Richard Kimberly Heck a écrit : On 7/23/24 5:49 PM, Jean-Marc Lasgouttes wrote: Le 23/07/2024 à 19:52, Jean-Marc Lasgouttes a écrit : commit a268fe096a58a7818554a7da4da85896bb289c81 Author: Jean-Marc Lasgouttes Date:   Tue Jul 23 15:04:49 2024 +0200 Fixup 216a6fb3

Re: [PATCH] WIP: rewrite statistics code

2024-07-24 Thread Jean-Marc Lasgouttes
Le 24/07/2024 à 09:43, Jean-Marc Lasgouttes a écrit : All in all this patch is nice piece of work! :) Thanks. It needs some cleanup, so I will push it later. I found the time, the code is now in. JMarc -- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mailman/listinfo

Re: [PATCH] WIP: rewrite statistics code

2024-07-24 Thread Jean-Marc Lasgouttes
Le 24/07/2024 à 12:30, Jean-Marc Lasgouttes a écrit : Le 24/07/2024 à 09:55, Pavel Sanda a écrit : On Wed, Jul 24, 2024 at 09:43:43AM +0200, Jean-Marc Lasgouttes wrote: Good, that was the point. The code I proposed to avoid running updateMacros on an unchanged document can be used here too

Re: LyX user file in linux

2024-07-24 Thread Jean-Marc Lasgouttes
Le 24/07/2024 à 15:36, Jürgen Spitzmüller a écrit : Am Mittwoch, dem 24.07.2024 um 10:40 +0100 schrieb José Matos: What do you think? Any idea about this? Since I do access it often, I am personally not too fond of having it hidden deeper down the hierarchy. I also think some external program

Re: [PATCH] WIP: rewrite statistics code

2024-07-24 Thread Jean-Marc Lasgouttes
Le 24/07/2024 à 09:55, Pavel Sanda a écrit : On Wed, Jul 24, 2024 at 09:43:43AM +0200, Jean-Marc Lasgouttes wrote: Good, that was the point. The code I proposed to avoid running updateMacros on an unchanged document can be used here too. Are updateMacros called for each caret movement in the

Re: [PATCH] WIP: rewrite statistics code

2024-07-24 Thread Jean-Marc Lasgouttes
Le 24 juillet 2024 01:57:01 GMT+02:00, Pavel Sanda a écrit : >I tested and do not see any counting difference in my selected document and >confirm that profiling gives about 3x better time with your patch. Thanks for testing. To be frank, I have been disappointed at first to have "only" a 3x s

Re: [LyX/master] Do not export the content of InsetMathBoxed as text in MathML

2024-07-23 Thread Jean-Marc Lasgouttes
Le 23/07/2024 à 23:48, Jean-Marc Lasgouttes a écrit : commit 1186d90edf9a6702573f296d5642c5e61ceaaedf Author: Jean-Marc Lasgouttes Date: Tue Jul 23 23:44:58 2024 +0200 Do not export the content of InsetMathBoxed as text in MathML \boxed{} is the only box in LaTeX that

Re: [LyX/master] Fixup 216a6fb348: close when quitting text mode

2024-07-23 Thread Jean-Marc Lasgouttes
Le 23/07/2024 à 22:11, Jean-Marc Lasgouttes a écrit : commit 1385f0fddd0229770496b1e50b76782892032d18 Author: Jean-Marc Lasgouttes Date: Tue Jul 23 22:02:50 2024 +0200 Fixup 216a6fb348: close when quitting text mode This fixes malformed documents on UserGuide export

Re: [LyX/master] Fixup 216a6fb3: fix broken xml syntax

2024-07-23 Thread Jean-Marc Lasgouttes
Le 23/07/2024 à 19:52, Jean-Marc Lasgouttes a écrit : commit a268fe096a58a7818554a7da4da85896bb289c81 Author: Jean-Marc Lasgouttes Date: Tue Jul 23 15:04:49 2024 +0200 Fixup 216a6fb3: fix broken xml syntax This commit addresses two issues: 1/ the embarassing one

Re: Many xhtml/docbook tests now failing on master

2024-07-23 Thread Jean-Marc Lasgouttes
Le 23/07/2024 à 22:22, Scott Kostyshak a écrit : On Tue, Jul 23, 2024 at 04:14:28PM GMT, Scott Kostyshak wrote: On Tue, Jul 23, 2024 at 10:12:32PM GMT, Jean-Marc Lasgouttes wrote: Le 23/07/2024 à 21:11, Scott Kostyshak a écrit : I believe this one is very recent: export/doc

Re: Many xhtml/docbook tests now failing on master

2024-07-23 Thread Jean-Marc Lasgouttes
Le 23/07/2024 à 21:11, Scott Kostyshak a écrit : I believe this one is very recent: export/doc/UserGuide_xhtml This corresponds to the English user guide. This one was easy. Do you have another one for me? JMarc -- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mailma

Re: Many xhtml/docbook tests now failing on master

2024-07-23 Thread Jean-Marc Lasgouttes
Le 23/07/2024 à 21:11, Kornel Benko a écrit : Am Tue, 23 Jul 2024 20:47:03 +0200 schrieb Jean-Marc Lasgouttes : Le 23/07/2024 à 20:23, Kornel Benko a écrit : From the 132 previously failed tests now are only 54 test failing. Some of the first are: 59% tests passed, 54 tests failed out of

Re: Many xhtml/docbook tests now failing on master

2024-07-23 Thread Jean-Marc Lasgouttes
Le 23/07/2024 à 20:53, Scott Kostyshak a écrit : I have 60 xhtml tests failing on current master now. Some are timing out so that might be a good place to start. For example, the following export times out for me after 10 minutes: cd lib/examples/Curricula_Vitae && lyx -e xhtml Modern_CV.lyx T

Re: Many xhtml/docbook tests now failing on master

2024-07-23 Thread Jean-Marc Lasgouttes
Le 23/07/2024 à 20:23, Kornel Benko a écrit : From the 132 previously failed tests now are only 54 test failing. Some of the first are: 59% tests passed, 54 tests failed out of 132 The following tests FAILED: 300 - export/export/docbook/basic_xhtml (Failed) 306 - export/export/do

Re: Many xhtml/docbook tests now failing on master

2024-07-23 Thread Jean-Marc Lasgouttes
Le 23/07/2024 à 19:24, Kornel Benko a écrit : Even with newest commits I get 71% tests passed, 132 tests failed out of 449 from # ctest -R .xhtml Kornel Hopefully most of the issues are gone now. The biggest and stupidest problem was that a backup variable was a bool instead o

Re: Many xhtml/docbook tests now failing on master

2024-07-23 Thread Jean-Marc Lasgouttes
Le 23/07/2024 à 19:24, Kornel Benko a écrit : Am Mon, 22 Jul 2024 16:47:14 -0400 schrieb Scott Kostyshak : I don't have time to look into them but a whole bunch of xhtml and docbook tests started failing recently on master. Kornel can you reproduce? Scott Even with newest commits I get 71% t

Re: Many xhtml/docbook tests now failing on master

2024-07-22 Thread Jean-Marc Lasgouttes
Le 22/07/2024 à 23:58, Scott Kostyshak a écrit : On Mon, Jul 22, 2024 at 11:27:46PM GMT, Jean-Marc Lasgouttes wrote: Le 22/07/2024 à 22:47, Scott Kostyshak a écrit : I don't have time to look into them but a whole bunch of xhtml and docbook tests started failing recently on master. Korne

Re: [LyX/2.4.x] Cleanup MathMLStream

2024-07-22 Thread Jean-Marc Lasgouttes
Le 22/07/2024 à 22:40, Enrico Forestieri a écrit : diff --git a/src/mathed/InsetMathChar.cpp b/src/mathed/InsetMathChar.cpp index 199982a5f7..82d0b63f1e 100644 --- a/src/mathed/InsetMathChar.cpp +++ b/src/mathed/InsetMathChar.cpp @@ -248,7 +248,7 @@ void InsetMathChar::mathmlize(MathMLStream & ms

Re: [LyX/2.4.x] Cleanup MathMLStream

2024-07-22 Thread Jean-Marc Lasgouttes
Le 22/07/2024 à 22:40, Enrico Forestieri a écrit : I don't understand the need for the two changes above. Note that on systems where sizeof(wchar_t) == 2, lyx::char_type is defined as a 32- bit value. So, with the above changes, char_ will be output as a number rather than a character. What a

Re: Many xhtml/docbook tests now failing on master

2024-07-22 Thread Jean-Marc Lasgouttes
Le 22/07/2024 à 22:47, Scott Kostyshak a écrit : I don't have time to look into them but a whole bunch of xhtml and docbook tests started failing recently on master. Kornel can you reproduce? I would be grateful to have individual examples. I guess the point made by Enrico is one of those issu

Re: [RFC] Make scrolling-by-selection smoother

2024-07-22 Thread Jean-Marc Lasgouttes
Le 22/07/2024 à 21:49, Pavel Sanda a écrit : On Sat, Jul 20, 2024 at 09:22:02PM +0200, Jean-Marc Lasgouttes wrote: So, especially if you have a slow-ish computer, please test. I tested on two archaic laptops and do not see any visible difference in speed. But the version with your patch is

Re: [PATCH] use yellow background for math comments

2024-07-21 Thread Jean-Marc Lasgouttes
Le 21/07/2024 à 09:49, José Matos a écrit : Actually it was more (for me a least) import that you applied one of the changes. Personally I prefer your first option but even the second option would be better than not to do nothing. I suppose that we can improve our communication skills and avoid

[PATCH] WIP: rewrite statistics code

2024-07-21 Thread Jean-Marc Lasgouttes
rom: Jean-Marc Lasgouttes Date: Sun, 21 Jul 2024 22:09:28 +0200 Subject: [PATCH] WIP: rewrite statistics code The statistics code is known to be very slow, because it relies on DocIterator to go through the buffer. This commit introduces a new Statitistics class that encapsulates the main code, al

Get rid of INSTALL.MacOSX?

2024-07-20 Thread Jean-Marc Lasgouttes
The last remaining bit of TODO.killqt4 is INSTALL.MacOSX. I thought we wanted to update it, but I fail to see what should be kept in this file as it is. Stephan, do we have a reason to keep it? Later, you could create a new INSTALL.macOS with up-to-date information if you feel that the file i

Re: [LyX/master] Fixup 2bdd691130: remove two more Update::SinglePar instances

2024-07-20 Thread Jean-Marc Lasgouttes
Le 20/07/2024 à 21:22, Richard Kimberly Heck a écrit : On 7/20/24 11:16 AM, Jean-Marc Lasgouttes wrote: Le 20/07/2024 à 17:13, Jean-Marc Lasgouttes a écrit : commit 485f5afc21ba553fd16465231de7570b0c631298 Author: Jean-Marc Lasgouttes Date:   Sat Jul 20 17:12:09 2024 +0200 Fixup

Re: [PATCH] use yellow background for math comments

2024-07-20 Thread Jean-Marc Lasgouttes
Le 19/06/2024 à 16:23, Jean-Marc Lasgouttes a écrit : I can't find the place where math comments appearance was discussed recently, so here is is. This trivial patch uses a yellow (Note) background to mark comments. While the result is not the same as Notes (the output actually goes to

Re: [RFC] Make scrolling-by-selection smoother

2024-07-20 Thread Jean-Marc Lasgouttes
Le 20/07/2024 à 21:23, Richard Kimberly Heck a écrit : On 7/20/24 3:22 PM, Jean-Marc Lasgouttes wrote: PS: the profiler tells me that more than 10% of this time is lost updating macros over and over. I'll take a look at that. That is at the root of a lot of the performance issues we

[RFC] Make scrolling-by-selection smoother

2024-07-20 Thread Jean-Marc Lasgouttes
I am not sure how. So, especially if you have a slow-ish computer, please test. JMarc PS: the profiler tells me that more than 10% of this time is lost updating macros over and over. I'll take a look at that.From 9da312f269b2ab3060d1b4231a82dc36787fd45a Mon Sep 17 00:00:00 2001 From: Jean

Re: [LyX/master] Fixup 2bdd691130: remove two more Update::SinglePar instances

2024-07-20 Thread Jean-Marc Lasgouttes
Le 20/07/2024 à 17:13, Jean-Marc Lasgouttes a écrit : commit 485f5afc21ba553fd16465231de7570b0c631298 Author: Jean-Marc Lasgouttes Date: Sat Jul 20 17:12:09 2024 +0200 Fixup 2bdd691130: remove two more Update::SinglePar instances They do not do what the author(s) of these

Re: SIGSEGV when doing undo after putting bib items in a note

2024-07-18 Thread Jean-Marc Lasgouttes
Le 16/07/2024 à 17:26, Richard Kimberly Heck a écrit : On 7/16/24 9:48 AM, Jean-Marc Lasgouttes wrote: Le 15/07/2024 à 18:14, Scott Kostyshak a écrit : I cannot reproduce, but the following patch makes sense. Does it work? Yes it fixes the behavior both with and without change tracking

Re: LyX 2.4.1 Released

2024-07-18 Thread Jean-Marc Lasgouttes
Le 18/07/2024 à 04:45, Richard Kimberly Heck a écrit : This line is present in master as well, but is actually coming from completely different changeset (1d1f95d2ed0b, JMarc, Reduce metrics updates from 4 to 1 when loading file) which is not part of 2.4. So it seems that it got accidentally i

Re: SIGSEGV when doing undo after putting bib items in a note

2024-07-16 Thread Jean-Marc Lasgouttes
it was kind of stable now. Riki, this is obviously candidate for branch. JMarc commit 82f387dabc2b18de6a9917192f4304e92bff8bc1 Author: Jean-Marc Lasgouttes Date: Tue Jul 16 15:43:11 2024 +0200 Fix faulty undo recording The recordUndo() call is very wrong, since we may be inserting

Re: Compilation warning

2024-07-15 Thread Jean-Marc Lasgouttes
Le 12/07/2024 à 21:25, Richard Kimberly Heck a écrit : On 7/12/24 11:13 AM, Jean-Marc Lasgouttes wrote: Le 12/07/2024 à 17:02, Jürgen Spitzmüller a écrit : Am Freitag, dem 12.07.2024 um 16:18 +0200 schrieb Jean-Marc Lasgouttes: It is the fault of autoconf 2.72, actually. Try the patch below

Re: SIGSEGV when doing undo after putting bib items in a note

2024-07-15 Thread Jean-Marc Lasgouttes
Le 02/07/2024 à 18:01, Scott Kostyshak a écrit : To reproduce: 1. Fresh build on current master. 2. Open User Guide. 3. Make sure that editing is enabled, and turn off change tracking. 4. Scroll to the end. 5. Select multiple bib items. 6. Click on the yellow note. 7. Do Undo with ctrl + z. I g

  1   2   3   4   5   6   7   8   9   10   >