On Thu, Feb 09, 2006 at 11:49:58PM +, Angus Leeming wrote:
Lars Gullik Bjønnes [EMAIL PROTECTED] writes:
I'll go into non-grumpy-mode after 1.4.0 is released (and when I have
this subversion thing up and going)
At least I will do my best to.
That'd be a shame. I quite enjoyed grumpy.
On Thu, Feb 09, 2006 at 11:56:56PM +0100, Abdelrazak Younes wrote:
Andre Poenitz a écrit :
-class ParagraphList : public std::vectorParagraph
+using namespace std;
Please no 'using' in headers.
Sure, this was just a quickly made proof of concept.
- ParagraphList::const_iterator p =
On Fri, Feb 10, 2006 at 03:34:52PM +0100, Jean-Marc Lasgouttes wrote:
Since our iterators store offsets (the pit, which are more stable than
a real iterator), we need direct access.
I think we need both interfaces.
*nod*
Andre'
On Thu, Feb 09, 2006 at 09:57:57PM +0100, Andre' Poenitz wrote:
iterator based interface. While it might be more clumsy to us,
s/us,/use,/
Andre'
On Thu, Feb 09, 2006 at 11:49:58PM +, Angus Leeming wrote:
> Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes:
> > I'll go into non-grumpy-mode after 1.4.0 is released (and when I have
> > this subversion thing up and going)
> > At least I will do my best to.
>
> That'd be a shame. I quite
On Thu, Feb 09, 2006 at 11:56:56PM +0100, Abdelrazak Younes wrote:
> Andre Poenitz a écrit :
> >>-class ParagraphList : public std::vector
> >>+using namespace std;
> >
> >Please no 'using' in headers.
>
> Sure, this was just a quickly made proof of concept.
>
> >>-
On Fri, Feb 10, 2006 at 03:34:52PM +0100, Jean-Marc Lasgouttes wrote:
> Since our iterators store offsets (the pit, which are more stable than
> a real iterator), we need direct access.
>
> I think we need both interfaces.
*nod*
Andre'
On Thu, Feb 09, 2006 at 09:57:57PM +0100, Andre' Poenitz wrote:
> iterator based interface. While it might be more clumsy to us,
s/us,/use,/
Andre'
Angus Leeming a écrit :
Ah... This is because I didn't go to the full monty... I forgot to erase
those three lines:
- advance(first, first_pit);
- ParagraphList::const_iterator last = plist.begin();
- advance(last, last_pit + 1);
It is better now,
Abdelrazak Younes a écrit :
Angus Leeming a écrit :
Ah... This is because I didn't go to the full monty... I forgot to
erase those three lines:
- advance(first, first_pit);
- ParagraphList::const_iterator last = plist.begin();
- advance(last, last_pit + 1);
It is
Abdelrazak Younes [EMAIL PROTECTED] writes:
By the way, right now, there are multiple instances in the code where
operator[] access is used. Thus implying that we must present a position
type interface (vector alike). At the same time, like in the case above,
there are instances where the
Angus == Angus Leeming [EMAIL PROTECTED] writes:
Angus Abdelrazak Younes [EMAIL PROTECTED] writes:
By the way, right now, there are multiple instances in the code
where operator[] access is used. Thus implying that we must present
a position type interface (vector alike). At the same time,
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes:
Abdelrazak By the way, right now, there are multiple instances in the
Abdelrazak code where operator[] access is used. Thus implying that
Abdelrazak we must present a position type interface (vector alike).
Abdelrazak At the same time,
Angus Leeming a écrit :
Ah... This is because I didn't go to the full monty... I forgot to erase
those three lines:
- advance(first, first_pit);
- ParagraphList::const_iterator last = plist.begin();
- advance(last, last_pit + 1);
It is better now,
Abdelrazak Younes a écrit :
Angus Leeming a écrit :
Ah... This is because I didn't go to the full monty... I forgot to
erase those three lines:
- advance(first, first_pit);
- ParagraphList::const_iterator last = plist.begin();
- advance(last, last_pit + 1);
It is
Abdelrazak Younes <[EMAIL PROTECTED]> writes:
> By the way, right now, there are multiple instances in the code where
> operator[] access is used. Thus implying that we must present a position
> type interface (vector alike). At the same time, like in the case above,
> there are instances where
> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes:
Angus> Abdelrazak Younes <[EMAIL PROTECTED]> writes:
>> By the way, right now, there are multiple instances in the code
>> where operator[] access is used. Thus implying that we must present
>> a position type interface (vector alike). At
> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
Abdelrazak> By the way, right now, there are multiple instances in the
Abdelrazak> code where operator[] access is used. Thus implying that
Abdelrazak> we must present a position type interface (vector alike).
Abdelrazak> At the
Andre Poenitz a écrit :
On Mon, Feb 06, 2006 at 09:59:42AM +0100, Jean-Marc Lasgouttes wrote:
Helge == Helge Hafting [EMAIL PROTECTED] writes:
Helge I tested the userguide, no problems there. Further testing
Helge shows that it is related to images. I have some high resolution
Helge images -
Abdelrazak Younes a écrit :
Andre Poenitz a écrit :
On Mon, Feb 06, 2006 at 09:59:42AM +0100, Jean-Marc Lasgouttes wrote:
Helge == Helge Hafting
[EMAIL PROTECTED] writes:
Helge I tested the userguide, no problems there. Further testing
Helge shows that it is related to images. I have some
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes:
Abdelrazak Hello Jean-Marc, maybe this is the bottleneck I am looking
Abdelrazak for. With my patch, undo/redo paragraph insertion is
Abdelrazak instantaneous and I can see that ParagraphList::insert
Abdelrazak and/or erase are called.
On Thu, Feb 09, 2006 at 12:25:41PM +0100, Abdelrazak Younes wrote:
Abdelrazak Younes a écrit :
...
Again when pasting multiple paragraphs, my debug info tells me that the
multiple insertion are instantaneous but I have to wait maybe 0.5 second
to see them on the screen...
Abdel.
Abdelrazak Younes a écrit :
On Mon, Feb 06, 2006 at 09:59:42AM +0100, Jean-Marc Lasgouttes wrote:
This is what I was about to suggest: currently, every time a paragraph
is created/killed, the whole document after it is duplicated. For
images, this means a lot of work. In particular, I think a
Jean-Marc Lasgouttes a écrit :
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes:
Abdelrazak Hello Jean-Marc, maybe this is the bottleneck I am looking
Abdelrazak for. With my patch, undo/redo paragraph insertion is
Abdelrazak instantaneous and I can see that ParagraphList::insert
Martin Vermeer a écrit :
On Thu, Feb 09, 2006 at 12:25:41PM +0100, Abdelrazak Younes wrote:
Abdelrazak Younes a écrit :
...
Again when pasting multiple paragraphs, my debug info tells me that the
multiple insertion are instantaneous but I have to wait maybe 0.5 second
to see them on the
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes:
Abdelrazak Abdelrazak Younes a écrit :
On Mon, Feb 06, 2006 at 09:59:42AM +0100, Jean-Marc Lasgouttes
wrote:
This is what I was about to suggest: currently, every time a
paragraph is created/killed, the whole document after it is
Abdelrazak Younes a écrit :
Martin Vermeer a écrit :
On Thu, Feb 09, 2006 at 12:25:41PM +0100, Abdelrazak Younes wrote:
Abdelrazak Younes a écrit :
...
Again when pasting multiple paragraphs, my debug info tells me that
the multiple insertion are instantaneous but I have to wait maybe 0.5
Jean-Marc Lasgouttes a écrit :
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes:
Abdelrazak Abdelrazak Younes a écrit :
On Mon, Feb 06, 2006 at 09:59:42AM +0100, Jean-Marc Lasgouttes
wrote:
This is what I was about to suggest: currently, every time a
paragraph is created/killed, the
Abdelrazak Younes a écrit :
Jean-Marc Lasgouttes a écrit :
Abdelrazak == Abdelrazak Younes
[EMAIL PROTECTED]
writes:
Abdelrazak Hello Jean-Marc, maybe this is the bottleneck I am looking
Abdelrazak for. With my patch, undo/redo paragraph insertion is
Abdelrazak instantaneous and I can see
Abdelrazak Younes [EMAIL PROTECTED] writes:
| I have put convenience function so as to minimize the changes needed
| in the code that uses ParagraphList. But there are some minimal (IMHO)
| change. The interface is simpler so a lot of code could be simplified
| with this new class.
By using a
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes:
But eraseIntern is only called when removing characters from a
paragraph, right?
Abdelrazak You are right of course. My use case is to type Enter in
Abdelrazak the middle of the paragraph of a big document (Extended or
Abdelrazak
Lars Gullik Bjønnes a écrit :
Abdelrazak Younes [EMAIL PROTECTED] writes:
| I have put convenience function so as to minimize the changes needed
| in the code that uses ParagraphList. But there are some minimal (IMHO)
| change. The interface is simpler so a lot of code could be simplified
|
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes:
IMHO first step is to make ParagraphList a proper class that
_contains_ a (private) vector. Methods should be added to this
ParagraphList so that what we use from the vector is covered.
Nothing more. No tricks with using etc. should
Jean-Marc Lasgouttes a écrit :
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes:
But eraseIntern is only called when removing characters from a
paragraph, right?
Abdelrazak You are right of course. My use case is to type Enter in
Abdelrazak the middle of the paragraph of a big
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes:
Abdelrazak I don't remember! I cannot configure since the automake
Abdelrazak changes plus I don't want to mess up my the Makefile that
Abdelrazak I have modified for the qt4 frontend. A typical gcc call
Abdelrazak is:
Abdelrazak g++
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes:
Abdelrazak The effect is the same for any paragraph inside
Abdelrazak Extended.lyx, but it seems that I was mis leaded by my
Abdelrazak debug info, sorry for the noise. Actually I think that it
Abdelrazak is the instructions just after
Jean-Marc == Jean-Marc Lasgouttes [EMAIL PROTECTED] writes:
Jean-Marc Wouldn't it be simpler to use a vector and change its
Jean-Marc interface to look like a vector? I am not sure why we want
Jean-Marc a list, actually. Inserting/erasing in a vector of pointers
Jean-Marc should be fast
Jean-Marc Lasgouttes a écrit :
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes:
IMHO first step is to make ParagraphList a proper class that
_contains_ a (private) vector. Methods should be added to this
ParagraphList so that what we use from the vector is covered.
Nothing more. No
Jean-Marc Lasgouttes a écrit :
Jean-Marc == Jean-Marc Lasgouttes [EMAIL PROTECTED] writes:
Jean-Marc Wouldn't it be simpler to use a vector and change its
Jean-Marc interface to look like a vector? I am not sure why we want
Jean-Marc a list, actually. Inserting/erasing in a vector of pointers
Abdelrazak Younes [EMAIL PROTECTED] writes:
| Lars Gullik Bjønnes a écrit :
| Abdelrazak Younes [EMAIL PROTECTED] writes:
| | I have put convenience function so as to minimize the changes
| needed
| | in the code that uses ParagraphList. But there are some minimal (IMHO)
| | change. The
Jean-Marc Lasgouttes a écrit :
Abdelrazak Again when pasting multiple paragraphs, my debug info
Abdelrazak tells me that the multiple insertion are instantaneous but
Abdelrazak I have to wait maybe 0.5 second to see them on the
Abdelrazak screen...
Could it be updateCounters()? This scans the
Abdelrazak Younes wrote:
This is why I had to write it++ instead of it+1 in some cases.
Just browing an interesting thread. You're doing great things here Abdel!
I highlighted the line above because language issues require less thought
than a real code review ;-)
++it is generally preferable
Angus Leeming a écrit :
Abdelrazak Younes wrote:
This is why I had to write it++ instead of it+1 in some cases.
Just browing an interesting thread. You're doing great things here Abdel!
Thanks. But I have to convince Lars...
I highlighted the line above because language issues require
OK Lars,
Why not going the other way? I can first provide a patch that will fix
interface problems like for example the '++it' instead of 'it+1' but
without touching ParagraphList.
Then a second patch will introduce the new ParagraphList (the full dual
vector/list one) that will provide the
On Thu, Feb 09, 2006 at 05:16:10PM +0100, Abdelrazak Younes wrote:
Jean-Marc Lasgouttes a écrit :
Abdelrazak Again when pasting multiple paragraphs, my debug info
Abdelrazak tells me that the multiple insertion are instantaneous but
Abdelrazak I have to wait maybe 0.5 second to see them on the
Martin Vermeer a écrit :
On Thu, Feb 09, 2006 at 05:16:10PM +0100, Abdelrazak Younes wrote:
Jean-Marc Lasgouttes a écrit :
Abdelrazak Again when pasting multiple paragraphs, my debug info
Abdelrazak tells me that the multiple insertion are instantaneous but
Abdelrazak I have to wait maybe 0.5
Abdelrazak Younes a écrit :
Martin Vermeer a écrit :
On Thu, Feb 09, 2006 at 05:16:10PM +0100, Abdelrazak Younes wrote:
Jean-Marc Lasgouttes a écrit :
Abdelrazak Again when pasting multiple paragraphs, my debug info
Abdelrazak tells me that the multiple insertion are instantaneous but
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes:
Abdelrazak As a side note, I don't really like this ParIterator class
Abdelrazak and I think most of its use could be replaced by a simple
Abdelrazak use of ParagraphList::iterator. But maybe it's just that I
Abdelrazak don't understand
Angus Leeming [EMAIL PROTECTED] writes:
| Abdelrazak Younes wrote:
|
| This is why I had to write it++ instead of it+1 in some cases.
|
| Just browing an interesting thread. You're doing great things here Abdel!
|
| I highlighted the line above because language issues require less thought
|
Abdelrazak Younes [EMAIL PROTECTED] writes:
| OK Lars,
|
| Why not going the other way? I can first provide a patch that will fix
| interface problems like for example the '++it' instead of 'it+1' but
| without touching ParagraphList.
| Then a second patch will introduce the new ParagraphList
On Thu, Feb 09, 2006 at 02:41:20PM +0100, Abdelrazak Younes wrote:
I think this is because the member fontlist is a std::vectorFontTable
and eraseIntern seems to be doing a lot of deletion. Would it make sense
to apply the same strategy as ParagraphList? It should be as easy as
templatify my
On Thu, Feb 09, 2006 at 11:58:26AM +0100, Abdelrazak Younes wrote:
Anyway, please find attached the patch. Everything but CutAndPaste
(which crashes lyx) seems to work OK. But I didn't do extensive
testing... sorry about that. If this is interesting to you, I'll try to
find some time this
Andre Poenitz [EMAIL PROTECTED] writes:
| Use 100 or so for VECTOR_RESERVE _at max_. In fact, I do think
| we should not reserve anything.
agree.
| I'd guess most of these functions should be inline in the header.
Not in the beginning IMHO, it is easy to play with implementation when
it is in
Abdelrazak Younes [EMAIL PROTECTED] writes:
I would be very happy if you could review my code.
One thing I like about your code is that it makes the code of clients of
ParagraphList a lot more readable. One place where this isn't the case is in
undo.C where code like this suggests that you
Angus Leeming [EMAIL PROTECTED] writes:
| Incidentally, there are two main differences between the way Lars and
| I review code:
| 1. I'm less grumpy and more encouraging
| 2. My opinion counts less than Lars'
I'll go into non-grumpy-mode after 1.4.0 is released (and when I have
this subversion
Lars Gullik Bjønnes a écrit :
Abdelrazak Younes [EMAIL PROTECTED] writes:
| OK Lars,
|
| Why not going the other way? I can first provide a patch that will fix
| interface problems like for example the '++it' instead of 'it+1' but
| without touching ParagraphList.
| Then a second patch will
Andre Poenitz a écrit :
-class ParagraphList : public std::vectorParagraph
+using namespace std;
Please no 'using' in headers.
Sure, this was just a quickly made proof of concept.
- ParagraphList::const_iterator p = par + 1;
+ ParagraphList::const_iterator p = par; p++;
Or 'p
Angus Leeming a écrit :
Abdelrazak Younes [EMAIL PROTECTED] writes:
I would be very happy if you could review my code.
One thing I like about your code is that it makes the code of clients of
ParagraphList a lot more readable. One place where this isn't the case is in
undo.C where code like
Abdelrazak Younes [EMAIL PROTECTED] writes:
One thing I like about your code is that it makes the code of clients of
ParagraphList a lot more readable. One place where this isn't the case is in
undo.C where code like this suggests that you haven't yet hit on the perfect
interface.
---
Abdelrazak Younes [EMAIL PROTECTED] writes:
+{
+ bool result=true;
+ for (size_t i=start; iend; ++i)
bool result = true;
for (size_t i = start; i end; ++i)
People are picky here ;-}
That picky?
Even pickier.
A good idiom to follow in for loops is to use
for
Lars Gullik Bjønnes [EMAIL PROTECTED] writes:
I'll go into non-grumpy-mode after 1.4.0 is released (and when I have
this subversion thing up and going)
At least I will do my best to.
That'd be a shame. I quite enjoyed grumpy.
Angus
Andre Poenitz a écrit :
On Mon, Feb 06, 2006 at 09:59:42AM +0100, Jean-Marc Lasgouttes wrote:
"Helge" == Helge Hafting <[EMAIL PROTECTED]> writes:
Helge> I tested the userguide, no problems there. Further testing
Helge> shows that it is related to images. I have some high resolution
Helge>
Abdelrazak Younes a écrit :
Andre Poenitz a écrit :
On Mon, Feb 06, 2006 at 09:59:42AM +0100, Jean-Marc Lasgouttes wrote:
"Helge" == Helge Hafting
<[EMAIL PROTECTED]> writes:
Helge> I tested the userguide, no problems there. Further testing
Helge> shows that it is related to images. I have
> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
Abdelrazak> Hello Jean-Marc, maybe this is the bottleneck I am looking
Abdelrazak> for. With my patch, undo/redo paragraph insertion is
Abdelrazak> instantaneous and I can see that ParagraphList::insert
Abdelrazak> and/or erase
On Thu, Feb 09, 2006 at 12:25:41PM +0100, Abdelrazak Younes wrote:
> Abdelrazak Younes a écrit :
...
> Again when pasting multiple paragraphs, my debug info tells me that the
> multiple insertion are instantaneous but I have to wait maybe 0.5 second
> to see them on the screen...
>
> Abdel.
Abdelrazak Younes a écrit :
On Mon, Feb 06, 2006 at 09:59:42AM +0100, Jean-Marc Lasgouttes wrote:
This is what I was about to suggest: currently, every time a paragraph
is created/killed, the whole document after it is duplicated. For
images, this means a lot of work. In particular, I think a
Jean-Marc Lasgouttes a écrit :
"Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
Abdelrazak> Hello Jean-Marc, maybe this is the bottleneck I am looking
Abdelrazak> for. With my patch, undo/redo paragraph insertion is
Abdelrazak> instantaneous and I can see that
Martin Vermeer a écrit :
On Thu, Feb 09, 2006 at 12:25:41PM +0100, Abdelrazak Younes wrote:
Abdelrazak Younes a écrit :
...
Again when pasting multiple paragraphs, my debug info tells me that the
multiple insertion are instantaneous but I have to wait maybe 0.5 second
to see them on the
> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
Abdelrazak> Abdelrazak Younes a écrit :
On Mon, Feb 06, 2006 at 09:59:42AM +0100, Jean-Marc Lasgouttes
wrote:
> This is what I was about to suggest: currently, every time a
> paragraph is created/killed, the
Abdelrazak Younes a écrit :
Martin Vermeer a écrit :
On Thu, Feb 09, 2006 at 12:25:41PM +0100, Abdelrazak Younes wrote:
Abdelrazak Younes a écrit :
...
Again when pasting multiple paragraphs, my debug info tells me that
the multiple insertion are instantaneous but I have to wait maybe 0.5
Jean-Marc Lasgouttes a écrit :
"Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
Abdelrazak> Abdelrazak Younes a écrit :
On Mon, Feb 06, 2006 at 09:59:42AM +0100, Jean-Marc Lasgouttes
wrote:
This is what I was about to suggest: currently, every time a
paragraph is created/killed,
Abdelrazak Younes a écrit :
Jean-Marc Lasgouttes a écrit :
"Abdelrazak" == Abdelrazak Younes
<[EMAIL PROTECTED]>
writes:
Abdelrazak> Hello Jean-Marc, maybe this is the bottleneck I am looking
Abdelrazak> for. With my patch, undo/redo paragraph insertion is
Abdelrazak> instantaneous and I can
Abdelrazak Younes <[EMAIL PROTECTED]> writes:
| I have put convenience function so as to minimize the changes needed
| in the code that uses ParagraphList. But there are some minimal (IMHO)
| change. The interface is simpler so a lot of code could be simplified
| with this new class.
By using a
> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
>> But eraseIntern is only called when removing characters from a
>> paragraph, right?
Abdelrazak> You are right of course. My use case is to type "Enter" in
Abdelrazak> the middle of the paragraph of a big document (Extended or
Lars Gullik Bjønnes a écrit :
Abdelrazak Younes <[EMAIL PROTECTED]> writes:
| I have put convenience function so as to minimize the changes needed
| in the code that uses ParagraphList. But there are some minimal (IMHO)
| change. The interface is simpler so a lot of code could be simplified
|
> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
>> IMHO first step is to make ParagraphList a proper class that
>> _contains_ a (private) vector. Methods should be added to this
>> ParagraphList so that what we use from the vector is covered.
>> Nothing more. No tricks with
Jean-Marc Lasgouttes a écrit :
"Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
But eraseIntern is only called when removing characters from a
paragraph, right?
Abdelrazak> You are right of course. My use case is to type "Enter" in
Abdelrazak> the middle of the paragraph of a
> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
Abdelrazak> I don't remember! I cannot configure since the automake
Abdelrazak> changes plus I don't want to mess up my the Makefile that
Abdelrazak> I have modified for the qt4 frontend. A typical gcc call
Abdelrazak> is:
> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
Abdelrazak> The effect is the same for any paragraph inside
Abdelrazak> "Extended.lyx", but it seems that I was mis leaded by my
Abdelrazak> debug info, sorry for the noise. Actually I think that it
Abdelrazak> is the
> "Jean-Marc" == Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes:
Jean-Marc> Wouldn't it be simpler to use a vector and change its
Jean-Marc> interface to look like a vector? I am not sure why we want
Jean-Marc> a list, actually. Inserting/erasing in a vector of pointers
Jean-Marc> should be
Jean-Marc Lasgouttes a écrit :
"Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
IMHO first step is to make ParagraphList a proper class that
_contains_ a (private) vector. Methods should be added to this
ParagraphList so that what we use from the vector is covered.
Nothing more.
Jean-Marc Lasgouttes a écrit :
"Jean-Marc" == Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes:
Jean-Marc> Wouldn't it be simpler to use a vector and change its
Jean-Marc> interface to look like a vector? I am not sure why we want
Jean-Marc> a list, actually. Inserting/erasing in a vector of
Abdelrazak Younes <[EMAIL PROTECTED]> writes:
| Lars Gullik Bjønnes a écrit :
| > Abdelrazak Younes <[EMAIL PROTECTED]> writes:
| > | I have put convenience function so as to minimize the changes
| > needed
| > | in the code that uses ParagraphList. But there are some minimal (IMHO)
| > | change.
Jean-Marc Lasgouttes a écrit :
Abdelrazak> Again when pasting multiple paragraphs, my debug info
Abdelrazak> tells me that the multiple insertion are instantaneous but
Abdelrazak> I have to wait maybe 0.5 second to see them on the
Abdelrazak> screen...
Could it be updateCounters()? This scans
Abdelrazak Younes wrote:
> This is why I had to write "it++" instead of "it+1" in some cases.
Just browing an interesting thread. You're doing great things here Abdel!
I highlighted the line above because language issues require less thought
than a real code review ;-)
"++it" is generally
Angus Leeming a écrit :
Abdelrazak Younes wrote:
This is why I had to write "it++" instead of "it+1" in some cases.
Just browing an interesting thread. You're doing great things here Abdel!
Thanks. But I have to convince Lars...
I highlighted the line above because language issues
OK Lars,
Why not going the other way? I can first provide a patch that will fix
interface problems like for example the '++it' instead of 'it+1' but
without touching ParagraphList.
Then a second patch will introduce the new ParagraphList (the full dual
vector/list one) that will provide the
On Thu, Feb 09, 2006 at 05:16:10PM +0100, Abdelrazak Younes wrote:
> Jean-Marc Lasgouttes a écrit :
> >Abdelrazak> Again when pasting multiple paragraphs, my debug info
> >Abdelrazak> tells me that the multiple insertion are instantaneous but
> >Abdelrazak> I have to wait maybe 0.5 second to see
Martin Vermeer a écrit :
On Thu, Feb 09, 2006 at 05:16:10PM +0100, Abdelrazak Younes wrote:
Jean-Marc Lasgouttes a écrit :
Abdelrazak> Again when pasting multiple paragraphs, my debug info
Abdelrazak> tells me that the multiple insertion are instantaneous but
Abdelrazak> I have to wait maybe
Abdelrazak Younes a écrit :
Martin Vermeer a écrit :
On Thu, Feb 09, 2006 at 05:16:10PM +0100, Abdelrazak Younes wrote:
Jean-Marc Lasgouttes a écrit :
Abdelrazak> Again when pasting multiple paragraphs, my debug info
Abdelrazak> tells me that the multiple insertion are instantaneous but
> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:
Abdelrazak> As a side note, I don't really like this ParIterator class
Abdelrazak> and I think most of its use could be replaced by a simple
Abdelrazak> use of ParagraphList::iterator. But maybe it's just that I
Abdelrazak> don't
Angus Leeming <[EMAIL PROTECTED]> writes:
| Abdelrazak Younes wrote:
|
| > This is why I had to write "it++" instead of "it+1" in some cases.
|
| Just browing an interesting thread. You're doing great things here Abdel!
|
| I highlighted the line above because language issues require less
Abdelrazak Younes <[EMAIL PROTECTED]> writes:
| OK Lars,
|
| Why not going the other way? I can first provide a patch that will fix
| interface problems like for example the '++it' instead of 'it+1' but
| without touching ParagraphList.
| Then a second patch will introduce the new ParagraphList
On Thu, Feb 09, 2006 at 02:41:20PM +0100, Abdelrazak Younes wrote:
> I think this is because the member fontlist is a std::vector
> and eraseIntern seems to be doing a lot of deletion. Would it make sense
> to apply the same strategy as ParagraphList? It should be as easy as
> "templatify" my
On Thu, Feb 09, 2006 at 11:58:26AM +0100, Abdelrazak Younes wrote:
> Anyway, please find attached the patch. Everything but CutAndPaste
> (which crashes lyx) seems to work OK. But I didn't do extensive
> testing... sorry about that. If this is interesting to you, I'll try to
> find some time
Andre Poenitz <[EMAIL PROTECTED]> writes:
| Use 100 or so for VECTOR_RESERVE _at max_. In fact, I do think
| we should not reserve anything.
agree.
| I'd guess most of these functions should be inline in the header.
Not in the beginning IMHO, it is easy to play with implementation when
it is
Abdelrazak Younes <[EMAIL PROTECTED]> writes:
> I would be very happy if you could review my code.
One thing I like about your code is that it makes the code of clients of
ParagraphList a lot more readable. One place where this isn't the case is in
undo.C where code like this suggests that you
Angus Leeming <[EMAIL PROTECTED]> writes:
| Incidentally, there are two main differences between the way Lars and
| I review code:
| 1. I'm less grumpy and more encouraging
| 2. My opinion counts less than Lars'
I'll go into non-grumpy-mode after 1.4.0 is released (and when I have
this
Lars Gullik Bjønnes a écrit :
Abdelrazak Younes <[EMAIL PROTECTED]> writes:
| OK Lars,
|
| Why not going the other way? I can first provide a patch that will fix
| interface problems like for example the '++it' instead of 'it+1' but
| without touching ParagraphList.
| Then a second patch will
Andre Poenitz a écrit :
-class ParagraphList : public std::vector
+using namespace std;
Please no 'using' in headers.
Sure, this was just a quickly made proof of concept.
- ParagraphList::const_iterator p = par + 1;
+ ParagraphList::const_iterator p = par; p++;
Or 'p =
1 - 100 of 120 matches
Mail list logo