Re: msevior - r25523 - in abiword/trunk/src: text/fmt/xp wp/ap/xp wp/impexp/xp

On Thu, Jan 15, 2009 at 10:18 AM, J.M. Maurer <uwog@uwog.net> wrote:
> On Thu, 2009-01-15 at 10:04 +1100, Martin Sevior wrote:
>> >> - if(!s_bFirstDrawDone)
>> >> + if(!s_bFirstDrawDone && (iPageCount > 1))
>> >> {
>> >
>> > Shouldn't this be "iPageCount >= 1" ?
>> >
>>
>> No I want more than one fully layed out page before drawing. It's a
>> good heuristic that the layotu is stable enough to draw to screen and
>> doesn't take a notible amount of extra time.
>
> I asked it because of the case that there is only 1 page to layout, so
> "> 1" would never hold.
>

That's OK. It will load in no time anyway. This code shows the first
pages of the document *as it loads*. It is useful for big documents
only. Small documents load essentially instantly anyway.

Cheers

Martin

> Marc
>
>
Received on Thu Jan 15 01:20:42 2009