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 <div class="gmail_quote">2011/7/19 David Tardon <<a href="mailto:dtardon@redhat.com" target="_blank">dtardon@redhat.com</a>> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">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 <a href="mailto:LibreOffice@lists.freedesktop.org" target="_blank">LibreOffice@lists.freedesktop.org</a> <a href="http://lists.freedesktop.org/mailman/listinfo/libreoffice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libreoffice</a> </blockquote></div>