Hello Eike, Kohei,<br><br><div class="gmail_quote">2011/8/9 Kohei Yoshida <span dir="ltr">&lt;<a href="mailto:kohei.yoshida@gmail.com" target="_blank">kohei.yoshida@gmail.com</a>&gt;</span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div><div></div><div>On Tue, 2011-08-09 at 05:04 +0200, Markus Mohrhard wrote:<br>
&gt; Hello,<br>
&gt;<br>
&gt; this patch removes entries like #REF! A1:A3 for named ranges when the<br>
&gt; sheet referenced by the named range is no longer there. I implemented<br>
&gt; Regina&#39;s first suggestion because I don&#39;t see a way to implement the<br>
&gt; other version with our current formula api.<br>
&gt;<br>
&gt; All in all this should be quite a save fix since I only added checks<br>
&gt; that the ScTokenArray contains no error and that the reference is<br>
&gt; valid.<br>
<br>
</div></div>So..., I feel a bit uneasy modifying this for 3.4.<br>
<br>
First of all, there is a likelihood that the said invalid formula syntax<br>
may be affecting other areas where the formula expressions are written,<br>
and fixing just for the export of named expressions may not fix the core<br>
cause.<br></blockquote><div><br>True. I&#39;ll have a look at some other problems too.<br><br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex">


<br>
Also, as Eike hinted, modifying the export code would also affect our<br>
ability to import the expression that we generate.  We need to check<br>
that, and if need be, fix it there as well.<br></blockquote><div><br>The import works quite well. It shows #Ref! as reference which should be quite fine.<br><br>
</div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex">
<br>
In addition to that, even after fixing the import to adopt to the new<br>
grammar rule, we&#39;ll need to be able to import the old, invalid syntax<br>
since there are documents out there that contain expressions written<br>
that way.<br></blockquote><div><br>I did not change the import filter. This feature is already implemented because [#Ref!] seems to be a valid expression whereas the old expression was not. <br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex">


<br>
Given all this, I would feel a little more comfortable nailing this on<br>
master the right way, and *then* see if the change is safe enough to be<br>
backported to the 3.4 branch.<br></blockquote><div><br>No problem with this. I&#39;ll push it to master with some other clean-up soon.<br><br>Regards, Markus<br></div><br></div><br>