On 2013-03-15 10:30, julien2412 wrote:
Noel Grandin wrote
I would guess that someone meant to move line 742 inside the first "if"
statement, but only did half the job.
Hello Noël,
Perhaps I'm too sleepy this morning but I don't see the interest to
duplicate the "new" part. Any hint?
I think the code should read:
if( nLeft || nRight )
{
if( !pCurr->GetPortion() )
pCurr->SetPortion( new SwTxtPortion( *pCurr ) );
if( nLeft )
{
SwMarginPortion *pMarg = new SwMarginPortion( 0 );
pMarg->AddPrtWidth( nLeft );
pMarg->SetPortion( pCurr->GetPortion() );
pCurr->SetPortion( pMarg );
}
if( nRight )
{
SwMarginPortion *pMarg = new SwMarginPortion( 0 );
pMarg->AddPrtWidth( nRight );
pCurr->FindLastPortion()->Append( pMarg );
}
}
Disclaimer: http://www.peralex.com/disclaimer.html
_______________________________________________
LibreOffice mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice
