<p>Wow... a simple symbol mistake of the brain, all the includes seemed to have hxx extension so....</p>
<p>It complies now pulling a diff together.<br>
</p>
<div class="gmail_quote">On Oct 16, 2010 3:30 PM, &quot;René Kjellerup&quot; &lt;<a href="mailto:rk.katana.steel@gmail.com">rk.katana.steel@gmail.com</a>&gt; wrote:<br type="attribution">&gt; Okay I&#39;ve now a beginning of a build of chart2 and I&#39;ve added:<br>
&gt; #include &lt;sal/macros.h&gt;<br>&gt; To the file, but the compilation says that the file is not found,<br>&gt; So where do one add the sal depends to the module? (or simply the includes)<br>&gt; On Oct 14, 2010 10:11 PM, &quot;Michael Meeks&quot; &lt;<a href="mailto:michael.meeks@novell.com">michael.meeks@novell.com</a>&gt; wrote:<br>
&gt;&gt; Hi Rene,<br>&gt;&gt;<br>&gt;&gt; On Thu, 2010-10-14 at 19:05 +0200, René Kjellerup wrote:<br>&gt;&gt;&gt; I&#39;ve looked through the calc&#39;s chart2 source and translated and spell<br>&gt;&gt;&gt; corrected a few comments in the view code<br>
&gt;&gt;<br>&gt;&gt; Cool - nice work thanks for that; I pushed it.<br>&gt;&gt;<br>&gt;&gt;&gt; in the ChartItemPool I&#39;ve added the use of SAL_N_ELEMENTS(array) too<br>&gt;&gt;<br>&gt;&gt; Yep; the slight issue here though is that we need to re-arrange the<br>
&gt;&gt; code a bit. Your changes would work, but only if SCH_ATTR_START == 0 -<br>&gt;&gt; since the range is offset otherwise. It would work if we had:<br>&gt;&gt;<br>&gt;&gt; - for( i = SCHATTR_START; i &lt;= SCHATTR_END; i++ )<br>
&gt;&gt; + for( i = 0; i &lt;= SAL_N_ELEMENTS(pItemInfos); i++ )<br>&gt;&gt; {<br>&gt;&gt; - pItemInfos[i - SCHATTR_START]._nSID = 0;<br>&gt;&gt; + pItemInfos[i]._nSID = 0;<br>&gt;&gt;<br>&gt;&gt; etc. ;-) it could perhaps be re-factored like that if you want, but<br>
&gt;&gt; prolly best to compile/test it.<br>&gt;&gt;<br>&gt;&gt; Anyhow - thanks for your work ! looking forward to what you work on<br>&gt;&gt; next - is there any area of LibreOffice you&#39;re particularly interested<br>
&gt;&gt; in contributing to ?<br>&gt;&gt;<br>&gt;&gt; All the best,<br>&gt;&gt;<br>&gt;&gt; Michael.<br>&gt;&gt;<br>&gt;&gt; --<br>&gt;&gt; <a href="mailto:michael.meeks@novell.com">michael.meeks@novell.com</a> &lt;&gt;&lt;, Pseudo Engineer, itinerant idiot<br>
&gt;&gt;<br>&gt;&gt;<br></div>