[Libreoffice] Review Request: fdo#39023 for 3.4.* branch

[Markus Mohrhard]

Hello David,

I think the original version should be safe. Our formula api can't deal with
broken documents during import especially the oox import can't deal with
errors of any kinds in formulas. But a broken document won't create a crash
only a #Name entry in the cell that is not correct.

I know that this situation is not ideal but we can't change the formula api
at the moment.

Regards, Markus

2011/7/19 David Tardon <dtardon at redhat.com>

> On Tue, Jul 19, 2011 at 04:29:32PM +0530, Muthu Subramanian K wrote:
> > Hi,
> >
> > +        if( maCurrCell.mnSharedId >= 0 )
> > +        {
> > +            Reference< XFormulaTokens > xTokens( maCurrCell.mxCell,
> UNO_QUERY_THROW );
> > +            ExtCellFormulaContext aContext( *this, xTokens,
> maCurrCell.maAddress );
> > +            getSharedFormulas().setSharedFormulaCell( aContext,
> maCurrCell.mnSharedId );
> > +        }
>
> This may cause problems with (broken) documents with cells that have
> type different from shared but contain si attribute. I am giving my +1
> on the condition that
>
> +        if( maCurrCell.mnSharedId >= 0 )
>
> is changed to
>
> +        if( maCurrCell.mnFormulaType == XML_shared &&
> maCurrCell.mnSharedId >= 0 )
>
> (or that someone convinces me the scenario I described cannot happen :)
>
> D.
> _______________________________________________
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20110719/74b96ef6/attachment.htm>