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

From: J.M. Maurer <uwog_at_uwog.net>
Date: Wed Jan 14 2009 - 21:20:39 CET

On Wed, 2009-01-14 at 14:00 +0100, cvs@abisource.com wrote:
> + if(pContainer->getY() == -99999999)
> + continue ; // container is not yet placed

Magic numbers are bad: maybe an idea to make it a DEFINE somewhere?

> Modified: abiword/trunk/src/wp/ap/xp/ap_EditMethods.cpp
> ===================================================================
> --- abiword/trunk/src/wp/ap/xp/ap_EditMethods.cpp 2009-01-14
> 02:10:00 UTC (rev 25522)
> +++ abiword/trunk/src/wp/ap/xp/ap_EditMethods.cpp 2009-01-14
> 13:00:38 UTC (rev 25523)
> @@ -1637,7 +1637,7 @@
> pLayout->updateLayout();
> iPageCount = pLayout->countPages();
>
> - if(!s_bFirstDrawDone)
> + if(!s_bFirstDrawDone && (iPageCount > 1))
> {

Shouldn't this be "iPageCount >= 1" ?

Cheers!
  Marc
Received on Wed Jan 14 21:21:35 2009

This archive was generated by hypermail 2.1.8 : Wed Jan 14 2009 - 21:21:35 CET